-
-
Notifications
You must be signed in to change notification settings - Fork 79.1k
Hamburger icon (U+2630 TRIGRAM FOR HEAVEN) is always black on Samsung Galaxy S Android devices #20234
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Unfortunately I don't have an Android 6 device handy, but testing in Android 5.1 via Sauce Labs shows that it does use the correct color there (but seems to lack the glyph in question). I highly recommend that you open a Chrome bug about this via https://crbug.com/new |
both jsbin examples (with and without bootstrap) WFM in Chrome 51 / Android 6.0.1 on Nexus 5, including loading the correct glyph. don't think it's a bootstrap issue per se. |
I also have black bars (like @tomlutzenberger) with a Galaxy S5 (SM-G900T) on Kit-Kat 4.4.2, Chrome 51.0.2704.81 |
Same issue, have the same problem with S7 on 6.0.1 SM-G930W8 and Chrome 51.0.2704.81 and Chrome Dev 53.0.2782.9 I just try the second code without Bootstrap, the bars are black but the background is white in both, dev and none dev Chrome version! |
@juampi @TheBuzzer67 @acshef @tomlutzenberger |
So a Chrome/Samsung issue then? (I have 51.0.2704.81 on Android 6.0.1 on a Nexus and don't see it) |
I can't reproduce on |
@tomlutzenberger Yes, we've established that this is Android-specific (and potentially Samsung-specific). |
@patrickhlauke Also, the glyph not rendering on Android 5.1 is a (separate) problem, since we currently claim support for Android v5.0+. |
Still shows black on Chrome Dev 53.0.2782.9 and Android 6.0.1 SM-G920F Build/MMB29K. |
Still black with Chrome Dev 53.0.2782.9 on Android 4.4.2 Galaxy S5 (SM-G900T). And for what it's worth, I also tried disabling the pre-installed Chrome Samsung Support Library (although honestly nobody knows for sure what that does). |
It might be a font issue. Since I can't test anything, somebody should try switching fonts and see if that helps. It might not, but it's worth a try. |
Yeah, that sheen on the ☰ character is weird. Maybe Samsung has some unusual default font that uses emoji-ish rendering where the color is baked-in. @tomlutzenberger Since you offered, could you please USB debug and inspect to see what font it's using? The panel is at the bottom of the "Computed" tab of the "Elements" inspector: |
Just to summarize the data y'all have contributed so far, the bug has been observed on:
|
Screenshot from the GMail app with @cvrebert's reply to this issue. Our culprit makes an appearance again! Definitely smells of "emoji-ish rendering where the color is baked-in" |
and although there's often talk about how much better having an actual word like "Menu" is compared to hamburger icons...many devs will want it, and I do think that users are (getting) familiar with this convention. |
@patrickhlauke But due to the removement of the glyphicons in v4, I don't see why any kind of a menu icon is needed. If a developer wants an icon instead of the word "Menu", he/she should include some icon font like FontAwesome. |
we also use "X" glyphs for dismissible alerts etc...so it's not the only situation where a subtle icon is in play... |
an alternative (but slightly more involved) solution would be to switch to using SVG icons, of course... |
But this would mean more could to maintain. |
@patrickhlauke Good points. I have no objection to your improved U+2261 approach, or an SVG approach. |
Unicode has support for forcing the text variant of an emoji, using variation selectors. http://www.unicode.org/emoji/charts/emoji-variants.html For this case, we'd use Unfortunately, from what I've found only iOS supports this. (It did not fix the issue on my Samsung S5 w/ Chrome 51.0.2704.81) |
Strangely, I can't seem to find U+2630 TRIGRAM FOR HEAVEN in the vanilla Noto Color Emoji font. Maybe Samsung added some glyphs without changing the font's name? Or maybe I'm not searching correctly. |
What? 😕 |
When it renders correctly on my phone, it uses the font "Noto Sans Symbols". |
A note: Gmail for their mobile mail app, uses three bullets in a vertical line to represent menus in some instances. Added point for consideration. The Icon debate is a hot one to be sure, but icons (if they are well understood see:save icon) improves scan-ability, and fits better into a 44px square area then a word does. There are work arounds in CSS that don't even bring in the img ( or the "i" ) tag into use, so accessibility is partly covered. However, stand alone icons isn't a great idea either because not all images make the same sense and in a growing global reality.... may even be offensive to some folks from other cultures. ^.^ Wether it is a button or a link, if we use their native functions, it works for accessibility. If it's coded. Semantically. The visual is just icing on the cake. Maybe, having a "sr-only" option for icons/images, with the associated words, allowing designers to expose that or not as they choose, may be a better way to go? |
Why exactly did you dump the ol' reliable method of using 3 |
as v4 targets android 5.0+, it was time to move on to some more modern technologies (using |
I'm not a fan of it personally; Using a font results in it rendering differently (or not at all) on a bunch of devices unless you're prepared to download a font, using some SVG magic would help, but either of these options would just add to precious loading time. Meanwhile all the code required to make the 3 |
@yesiateyoursheep So you suggest to furthermore keep a method...
No offense, but IMHO in this context, it's like saying "Even if there are better and more modern ways to do it, don't change it anyway. |
@tomlutzenberger I do like the class support and animation with SVG, but I don't like how this is default, that's a pretty harsh price to pay for a critical part of the UI not rendering on some devices. Edit: Did a little research; Android 4.3 and below have partial or no support for SVGs, if devices still running >4.3 can handle running Chrome (in my case, Chrome was too slow on my old phone, so I didn't use it.) - That's 22.9% of Android users who might have invisible or even unclickable menu buttons. |
Partially, I agree. It's your responsibility as Developer, to make sure that your audience is capable to access the information of your website or your product they want and need. Don't shove this off to BS. |
Is there a way for developers/designers to change out the hamburger for something else? (like three vertically stacked dots, or wording)? |
You can use whatever you want. |
If we go the SVG route, there will be a Sass variable you can use to swap out the inline SVG with your own SVG image. |
Under the assumption that the Android Browser data on http://caniuse.com/#feat=svg also applies to Chrome when used with those Android versions, Android 3+ should support SVG (modulo masking, which shouldn't matter in our case). |
The triple bar / hamburger icon can be replaced with ≡ ['IDENTICAL TO' (U+2261) – Ed.] in the HTML code! |
@touficbatache Doesn't look quite right to mdo. See #20260 (comment) |
I get this on |
OS: Android 6.0.1 SM-G920F Build/MMB29K
Browser: Chrome 51.0.2704.81
The Bootstrap 4 navbar hamburger icon is always black, even when using other background colors such as bg-inverse or bg-primary.
The bug can be reproduced by accessing this bin:
https://output.jsbin.com/hiyeti/3
EDIT:
I just realized this behavior is present even without Bootstrap:
https://output.jsbin.com/wovuba/1
Probably an Android/Chrome issue then, though not sure if anything should be done about this on Bootstrap's side.
EDIT 2:

Here's a screenshot:
The text was updated successfully, but these errors were encountered: