19696 2012-10-25 05:16:49 +0000 Bottom part of logo doesn't link (masked by #sitenav .container) 2012-12-18 12:43:46 +0000 1 1 1 Unclassified webplatform.org skin unspecified All All RESOLVED FIXED P2 trivial --- 1 samdutton schepers Heath jonathan lea team-webplatform-admin public-webplatform-bugs oldest_to_newest 77086 0 samdutton 2012-10-25 05:16:49 +0000 The bottom part of the webplatform logo doesn't link because #sitenav .container is overlaid on it. Adding the following to #p-logo would fix the problem, but this is a bit of a hack: z-index: 1; position: relative; The problem might be better solved by adding margin-left to #sitenav .container and removing the padding-left from ul.links. 77700 1 jonathan 2012-11-01 21:16:17 +0000 *** Bug 19822 has been marked as a duplicate of this bug. *** 77701 2 jonathan 2012-11-01 21:17:11 +0000 Bug 19822 suggested another possible fix. I myself haven't had it work though. 77702 3 Heath 2012-11-01 21:20:16 +0000 Change: #sitenav .links { padding-left: 150px; } To: #sitenav .links { margin-left: 150px; } This will fix it without needing to bother with z-index or positions. 77703 4 Heath 2012-11-01 21:29:21 +0000 (In reply to comment #3) > Change: > #sitenav .links { > padding-left: 150px; > } > > To: > #sitenav .links { > margin-left: 150px; > } > > This will fix it without needing to bother with z-index or positions. Also, we would need to do the following to the mobile media query. @media screen and (max-width: 520px) #sitenav .links { padding: 0; } 77704 5 Heath 2012-11-01 21:30:26 +0000 (In reply to comment #4) > (In reply to comment #3) > > Change: > > #sitenav .links { > > padding-left: 150px; > > } > > > > To: > > #sitenav .links { > > margin-left: 150px; > > } > > > > This will fix it without needing to bother with z-index or positions. > > Also, we would need to do the following to the mobile media query. > > @media screen and (max-width: 520px) > #sitenav .links { > padding: 0; > } Sorry, misclicked. Also, we would need to do the following to the mobile media query. change: @media screen and (max-width: 520px) #sitenav .links { padding: 0; } to: @media screen and (max-width: 520px) #sitenav .links { padding: 0; margin: 0; } 80287 6 lea 2012-12-17 16:57:48 +0000 (In reply to comment #3) > Change: > #sitenav .links { > padding-left: 150px; > } > > To: > #sitenav .links { > margin-left: 150px; > } > > This will fix it without needing to bother with z-index or positions. This doesn’t seem to fix it. pointer-events: none would fix it really easily, but the fix won't (currently) work in Opera and IE. However, it’s minor enough that it might be ok. 80288 7 lea 2012-12-17 17:04:10 +0000 > pointer-events: none would fix it really easily, but the fix won't > (currently) work in Opera and IE. However, it’s minor enough that it might > be ok. Nah, my comment was nonsense. pointer-events: none; can't be used here. I think the z-index solution is the only one that seems to work, and comes with the minimum hassle. 80338 8 lea 2012-12-18 12:43:46 +0000 Fixed in the test wiki, so will propagate to stable once we deploy. For the record, the only thing that was needed was z-index: 1 on #p-logo.