Closed (outdated)
Project:
Drupal core
Version:
7.x-dev
Component:
Bartik theme
Priority:
Minor
Category:
Bug report
Assigned:
Reporter:
Created:
3 Jul 2010 at 21:46 UTC
Updated:
16 Apr 2024 at 15:35 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
jensimmons commentedEmbedding screenshots from above:
Normal:
Opera:
Comment #2
aspilicious commentedOpera in windows is doing fine...
What are we going to do with this edge case?
Comment #3
aspilicious commentedOpera in windows is doing fine...
What are we going to do with this edge case?
Comment #4
munzirtaha commentedHere is a small patch to fix the problem. Please review.
Comment #5
munzirtaha commentedComment #6
Jeff Burnz commented#903460: Bartik Tabs broken in Firefox may fix this issue, I could re-roll that patch so you can test it, I'd much rather a big patch like that goes in rather than these one liners which cause everything to be re-rolled.
Broken display is never minor - setting back to normal.
Comment #7
munzirtaha commentedThis bug is about one pixel off and it's different than the bug you pointed to. Beside having a trivial patch like this is more likely to get accepted than refactoring the tabs code, I believe. In all cases, please re-roll your patch and let's test it.
Comment #8
fgmProblem no longer happens with Opera 10.61 on Linux, on same URL: http://bartik.labs.treehouseagency.com/ , so it seems this patch is no longer needed. Maybe it was a bug in Presto, not Bartik;
Comment #9
Jeff Burnz commentedOK, thanks for the heads up - I think we can close this. Please open again if a problem persists.
FYI I have updated the patch at #903460: Bartik Tabs broken in Firefox
Comment #10
munzirtaha commentedI am using Opera 10.62 and the bug is still there in that same site and on a local site. I would re-open the bug and check Jeff Burnz patch for the other bug ...
Comment #11
Jeff Burnz commentedThe other patch changes the technique used to theme the main menu tabs and this patch will only be a band-aid - we need to fix the root of the problems. Please test the other patch and report your findings there - that is the way forward.
I am postponing this until after #903460: Bartik Tabs broken in Firefox gets in, if there's still a problem we can look at this again.
Comment #12
fgmI did further testing:
- Lucid + Opera "10.60 internal": the bug is there,
- Karmic + Opera 10.61, bug not there
- Lucid + Opera 10.62: bug there
Comment #13
munzirtaha commented@fgm: please test the last patch (13) in #903460: Bartik Tabs broken in Firefox
Comment #14
fgm@munzirtaha: does not change anything. Same issue.
I very much doubt it is related with #903460, though, especially since I am not using a RTL language, and the problem is vertical spacing, not horizontal.
1pixel.png is Lucid+Opera10.62, 0pixel is Lucid+Chrome6.0.472.63 beta
Comment #15
fgmOne more test, still on http://bartik.labs.treehouseagency.com/
- Maverick + Opera 10.63: bug there
Also, maybe more significantly, there is a lime-to-purple color gradient at that URL in Chrome and FF, but not in Opera where the header is full purple. This is actually much more visible than the 1-pixel issue, but I suspect it must have been noticed in another issue.
Comment #16
Jeff Burnz commentedCan you check again, there was a change to how core handles ul.links and a new patch for Bartik went in (it created a 1px problem for linux FF), if the header is a solid color then the browser likely does not support CSS gradients.
Comment #17
fgmNo change, at least on http://bartik.labs.treehouseagency.com/ : pixel still there.
I checked and Opera is indeed reported as not supporting CSS gradients in 10.x versions.
Comment #18
jensimmons commentedSetting this to minor in an effort to get some differentiation in the Bartik issue queue. (Having almost everything marked as normal makes Priority useless.)
Comment #19
avpadernoSince there is a patch, the status should be Needs review.
Comment #20
poker10 commentedI downloaded a newest Opera browser and opened Drupal 7.100 clean install on simplytest.me. I do not see this 1px offset on Bartik theme. I would say this is probably resolved after 13 years. Feel free to reopen if this is still an issue. Thanks!