Ticket #4290 (closed defect: wontfix)

Opened 3 years ago

Last modified 2 years ago

Make CSS selectable

Reported by: izzy Assigned to: cbalan
Priority: normal Component: MenusPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.11

Description

I think it would be a good idea to make the CSS to use selectable, so one has not to nullify existing settings before applying own ones. I just created a patch for this: it adds a new option to the [menu-custom] section of the trac.ini - menu_style. Here one can either select from the "shipped" CSS files (with my patch: 'simple' (the current one), 'superfish' (the one used previously), and 'trac' - the one I made), or define a complete URL for an own CSS file (which then is taken literally). 

[menu-custom]
menu_style = superfish

For this to work, the current superfish.css file has to be split to basic.css (holding the "essential" part) and simple.css (holding the "Demo Style"). Plus from the previous superfish.css, the "Demo Style" part should be added as superfish.css. If you lost my trac.css, I'll attach it again here (wait - I'll either attach it again later, there are some modifications I need to finish).

Hope you like and apply the patch ;)

All the best, Izzy.

Attachments

web_ui.py.css.patch (1.8 kB) - added by izzy on 12/16/08 17:28:17.
Adding stylesheet selection to trac.ini
trac.css (1.5 kB) - added by izzy on 12/16/08 17:42:58.
trac.css to make the menu looking more like the trac original

Change History

12/16/08 17:28:17 changed by izzy

  • attachment web_ui.py.css.patch added.

Adding stylesheet selection to trac.ini

12/16/08 17:42:58 changed by izzy

  • attachment trac.css added.

trac.css to make the menu looking more like the trac original

12/16/08 18:11:08 changed by cbalan

  • status changed from new to assigned.

Hey Izzy,

This is great, but personally I believe that any special customizations should be done using wiki:TracInterfaceCustomization#SiteAppearance "method".

The default css shipped with this plugin should only match trac default's theme. This is the reason why [menu-custom]serve_ui_files exists, to prevent the default css to be added, if there is a custom theme in place.

I do like this idea(your patch is great) but having in mind :

  1. the wiki:TracInterfaceCustomization
  2. for a custom theme, one css file can "cover everything", including menus and other plugins (plus is easy to maintain it)
  3. having files served from /static (custom theme case) would work better than /chrome/...

I consider this approach out of scope.

Thank you, Catalin Balan

12/16/08 19:47:19 changed by cbalan

(In [5021]) MenusPlugin: - More trac-alike look thanks to Izzy. refs #4290 #4264 #4274

09/15/09 18:29:15 changed by cbalan

  • status changed from assigned to closed.
  • resolution set to wontfix.

Add/Change #4290 (Make CSS selectable)




Change Properties
Action