Opened 14 years ago
Closed 13 years ago
#16901 closed enhancement (invalid)
Minor changes to theme install page
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | minor | Version: | 3.1 |
Component: | Themes | Keywords: | has-patch |
Focuses: | 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)
#2
in reply to:
↑ 1
@
14 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
@
14 years ago
- 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?
#4
@
14 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.
The case is that not all themes have the data I'd suggest.