Opened 2 years ago
Closed 11 months ago
#16901 closed enhancement (invalid)
Minor changes to theme install page
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | Themes | Version: | 3.1 |
| Severity: | minor | Keywords: | has-patch |
| Cc: |
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)
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.
- Owner set to scribu
- Status changed from new to reviewing
Please be more descriptive: What do these minor changes do? What are you trying to accomplish?
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.
The case is that not all themes have the data I'd suggest.