Closed (fixed)
Project:
Professional Theme
Version:
7.x-1.5
Component:
User interface
Priority:
Critical
Category:
Feature request
Assigned:
Reporter:
Created:
8 Apr 2013 at 19:55 UTC
Updated:
3 May 2013 at 22:40 UTC
Jump to comment: Most recent file
Comments
Comment #1
mheinke commentedwhat exactly are you changing?
im afraid i dont understand the issue?
Comment #2
mheinke commentedComment #3
ibou65 commentedHi I noticed some issues with IE7, even on your demo site.
1) Slideshow: there is a display issue with slide indicator (link that leads to a slide). The links are displayed vertically, not horizontally. CSS float issue ?
2) Main menu: submenus display behind the slideshow area (some links are hidden). z-index problem ?
3) Main menu: when I move the mouse to a link with sub link and I move the mouse to another link in the main menu, some little bubbles appear (the same bubble we can see with UL items).
All these issues are also on your demo site and with IE7. I see no problem with another browser.
Note: each issue is visible on the picture attached.
Comment #4
mheinke commentedok now i understand,
IE7 is not truely supported right now.
BUT, i will add IE7 support in the next release :)
if you have any CSS fixes that you are doing in the mean time please add them to this issue queue, i would love some help on this.
Comment #5
mheinke commentedComment #6
ibou65 commentedThere is one solution to the issues I discovered above with IE7.
#main-menu li {
position:relative;
float:left;
padding:0;
margin:0;
line-height:40px;
z-index:100;/** Added by ibou65 to fix the submenu displaying behind the slideshow area on IE7, line 406 **/
list-style-image:url(none);/** Added by ibou65 to prevent some little bubbles to appear when moving the mouse to a link with sub link and moving the mouse to another link in the main menu on IE7, line 407 **/
}
/** Added by ibou65 to fix margin space between submenu links in main menu on IE7. This issue is du to line-height:40px set above, line 409 **/
#main-menu li li{
line-height:13px;
}
.flex-control-nav li {
display:inline-block;
margin:0 0 0 15px;
float:left; /** Added by ibou65 to fix vertical display of slides link on IE7, line 603 **/
}
Note: I have made this change to the css file of your last release (professional_theme-7.x-1.7). So you can check if everything is right and replace the previous css file by this one.
Comment #7
mheinke commentedawesome!
thank you for the contribution.
i am adding a ie7.css file with this override in it
thank you!
Comment #8
mheinke commentedim creating a dev version right now with your updates (and i kept the comments in there as well that way it gives you proper attribution :) )
should be released tonight (PST)
Comment #9
mheinke commentedversion 1.8 adds your CSS fixes
Comment #10
mheinke commented