Menu

#1095 Change to the JS menu separator image

open-postponed
Veit
5
2009-07-19
2009-01-22
No

Currently the menu separator image in getMenu() is the horizontal line image. To allow greater flexibility for theme designers, this can be changed to an empty <div class="HR"> as it is in getMenuAsList(). This will not break any existing themes, but the separator will not show up unless the class is added to CSS.

The current line is in a DIV with the menu ID and other attributes but I'm not sure they are necessary.

This is around line 205 of class_menu.php.

Discussion

  • Greg Roach

    Greg Roach - 2009-06-13

    Veit - was this part of your recent changes?

     
  • Greg Roach

    Greg Roach - 2009-06-13
    • assigned_to: nobody --> volschin
     
  • Veit

    Veit - 2009-06-13

    I only changed the behavour for getMenuAsList() to use an <li class="separator">.
    I will check to do the same for the classic menu, means <div class="separator">. Difficulty will be not to have an empty div which can cause display problems.

     
  • Veit

    Veit - 2009-07-19

    This must be postponed, because I could not get it working with IE6 and IE7. Alle other browsers (IE8, FF3, GC, Safari4 and Opera) work fine. I also tried with a span tag between, but didn't help.

    At least I have moved the inline styles to the theme css and you can deal with them.

    If you have an idea to get it working with css also for these browsers I would like to implement it.

     
  • Veit

    Veit - 2009-07-19
    • status: open --> open-postponed
     

Log in to post a comment.

Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.