Opened 15 years ago
Closed 14 years ago
#16901 closed enhancement (invalid)
Minor changes to theme install page
| Reported by: | Simek | Owned by: | scribu |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | Themes | Version: | 3.1 |
| Severity: | minor | Keywords: | has-patch |
| Cc: | Focuses: |
Description
I've prepared minor changes to Theme Install page. Some basic stuff at the moment.
I've also got a question about "theme" object and his attributes. There are a lot of "!empty" conditionals for attributes but when I've used var_dump on "theme" part of the just don't exist. It's a mistake or I've miss something?
I would like also continue my work to unify UI/apperance of Manage Themes and Install Theme pages.
Attachments (1)
Change History (6)
#2
in reply to: ↑ 1
@
15 years ago
Replying to dd32:
There are a lot of "!empty" conditionals for attributes but when I've used var_dump on "theme" part of the just don't exist. It's a mistake or I've miss something?
The case is that not all themes have the data I'd suggest.
It's strange that Twenty Ten also don't have this data.
#3
@
15 years ago
- Owner set to
- Status new → reviewing
Please be more descriptive: What do these minor changes do? What are you trying to accomplish?
#4
@
15 years ago
At the moment patch introduce:
- better (in loop) themes stars rating generation
- provides modified way of display when theme was updated
- themes descriptions justified
My goal is to unify and improve usablity of the UI and way of content display for Installed Themes and Theme Browser.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
The case is that not all themes have the data I'd suggest.