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

Update popup design #105

neocotic opened this issue May 21, 2012 · 3 comments

Update popup design #105

neocotic opened this issue May 21, 2012 · 3 comments


Copy link

@neocotic neocotic commented May 21, 2012

Update the style of the popup to better match that of the options page and Bootstrap dropdown menus.

Additionally, this would probably look best if the Template icons were changed to use Glyphicons instead.

I'll probably have to adapter bootstrap's CSS since their selectors for dropdown menus won't match our use.

Copy link
Member Author

@neocotic neocotic commented May 23, 2012

The largest part of this change still remains, updating the icons. The following is an initial estimate of the tasks involved with this;

  • Fix icon lookups for templates
  • Fix icon selection on Options page
  • Change the default icons of predefined templates to better matching new icons (instead of just using their mapped counterparts)
    • See the end of this comment for suggested new default icons
  • Convert icons for existing templates to the new system
    • See the end of this comment for a preliminary mapping of icons
    • This should be done for predefined templates in and while updating existing templates for 1.1.0
    • During the 1.1.0 update, predefined templates still using their default icon should now use their new default icon instead of their existing icons new mapping
  • Delete all obsolete image files (including spacer and loading animation) and i18n messages
  • Add new i18n messages or change Options page icon selector to no longer require text descriptions
  • Remove functionality as the new icons are CSS driven and won't work (this feature isn't as nice as I first hoped anyway)
  • Fix import/export functionality
  • Update screenshots

I'm considering abstracting icon/image management out of and into its own script. I may not be able to abstract all functionality but this should hopefully make things cleaner and simpler.

Default Icons

PREDEFINED.00001 => globe
PREDEFINED.00002 => tag
PREDEFINED.00003 => font
PREDEFINED.00004 => lock
PREDEFINED.00005 => comment
PREDEFINED.00006 => asterisk
PREDEFINED.00007 => italic

Icon Mapping

tmpl_auction => shopping-cart
tmpl_bug => wrench
tmpl_clipboard => folder-open
tmpl_clipboard_empty => folder-close
tmpl_component => th-large
tmpl_cookies => certificate
tmpl_discussion => comment
tmpl_globe => globe
tmpl_google => briefcase
tmpl_heart => heart
tmpl_html => font
tmpl_key => lock
tmpl_lightbulb => star-empty
tmpl_lighthouse => road
tmpl_lightning => star
tmpl_link => tag
tmpl_linux => briefcase
tmpl_mail => envelope
tmpl_newspaper => bullhorn
tmpl_note => inbox
tmpl_page => file
tmpl_plugin => leaf
tmpl_rss => signal
tmpl_script => file
tmpl_scull => fire
tmpl_sign => briefcase
tmpl_siren => bell
tmpl_star => star
tmpl_support => question-sign
tmpl_tag => tag
tmpl_tags => tags
tmpl_thumb_down => thumbs-down
tmpl_thumb_up => thumbs-up
tmpl_tools => wrench
Copy link
Member Author

@neocotic neocotic commented May 24, 2012

I'm happy with the testing done so far and the popup looks great now, but I want to do some more as I really don't want to break anything after changing so much.

I also still need to update the screenshots which are now outdated but I'll do that later tonight and will hopefully release 1.1.0 after that or at some point tomorrow. Sadly Chrome has some strange resolution size requirements that happen to appear best on my Mac, at home.

Copy link
Member Author

@neocotic neocotic commented May 24, 2012

I've tested version updates and fresh installs and I see no problems.

I hope everyone enjoys the new uniformed look!

@neocotic neocotic closed this May 24, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.