#20147 closed enhancement (fixed)
Remove Tags listing from Manage Themes
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 3.4 | Priority: | normal |
Severity: | minor | Version: | 3.3 |
Component: | Themes | Keywords: | has-patch ui-feedback |
Focuses: | Cc: |
Description
Tags are listed under "Manage Themes" (currently installed), both for your current theme, and other themes installed (after clicking on "Details", with respect to 3.4).
Since they're generally only useful during a search for a theme, it's proposed that they be removed from view, as they are already not visible after a search in "Install Themes."
I ran this by Jane, and she agreed with the change.
Attachments (10)
Change History (21)
#1
@
13 years ago
20147.diff removes the tags from "Manage Themes" -- both from the current theme, and available themes listings.
Also removed a <br>, since the remaining margin between the theme options and "Available Themes" text was significant after removing tags.
#2
@
13 years ago
- Keywords ui-feedback added
Added preview imaged and additional options to choose from, in case the spacing isn't liked.
With input from mattfelten, created 20147.3.diff, which I prefer, and you can see previewed in 20147.3.jpg.
#4
@
13 years ago
Available Themes should still come before search in the source. I fiddled with this last week and a simple float plus change of the top margin of the h3 was enough to line things up. Perhaps <h3 class="available-themes"> then with the slightest bit of styling...
@
13 years ago
Align "Available Themes" header with search box via styles, rather than moving it below search.
#5
@
13 years ago
Attached 20147.4.diff, which aligns the "Available Themes" header with the search box via styles, rather than moving it below search.
Looks like the three options for styling this header include assigning a new ID or class, moving it inside the search-form form, or wrapping all available themes "stuff" in a div.
Smallest change would be the ID (along with proper clear on the form), as in 20147.4.diff, but if making "current theme" and "available theme" stuff enclosed in divs is acceptable, that seems a bit cleaner.
Any preferences?
#7
@
13 years ago
- Owner set to nacin
- Resolution set to fixed
- Status changed from new to closed
In [20065]:
#9
@
13 years ago
Looks like we need a little clearing action on #availablethemes
in the case of no results found, lest this happens. 20147.5.diff
Remove tags listing and extra <br> between tags/"Available Themes"