Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add background highlight to currently active theme in admin #155

Merged
merged 1 commit into from Feb 24, 2013

Conversation

randomecho
Copy link
Contributor

The italic text to show what theme is active is too small. This marks out the current theme in use with an alert DIV.

Tidy up the if-else block uses as the contents are mostly different between inactive choices and the active one.

"Use this theme" sounded more direct and action packed than "Choose this theme" for the button.

This also cleans up some language file definitions orphaned by removal of references to Typogarden and installing themes in commit 82c73cd

Instead of a small bit of italic text, mark out the current theme
in use with an alert DIV. Clearer, quick to see if you have many
themes to choose from.

Tidy up the if-else block uses as the contents are mostly different
between inactive choices and the active one.

"Use this theme" sounded more direct and action packed than
"Choose this theme" for the button.

Cleans up language file definitions orphaned by removal of
references to Typogarden and installing themes in
commit 82c73cd
@mvz mvz merged commit 352ee29 into publify:master Feb 24, 2013
@randomecho randomecho deleted the theme-selections branch February 24, 2013 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants