-
Notifications
You must be signed in to change notification settings - Fork 22
$15 | All Sub menus> Sub-menu has an item after the menu name #11
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
Please make sure to fix issue reported in #2438 too. (are related issues) |
PR #5 |
@nahidshahin Please check feedback in PR #5 |
@luizrrodrigues this task involves added Icon + place icons appropriately. The prize seems little bit small. |
I think is okay, second requirement is just add |
@nahidshahin Please check feedback here: #5 (review) |
@nahidshahin I do a double check in real device, wrong icon still showing in Android. |
@nahidshahin Please give me some status on this or if we can Open For Pickup |
@luizrrodrigues I would like to pick this up. Thanks. |
@r0hit-gupta Cool, thanks. |
@luizrrodrigues unable to reproduce. Can you please test again in Chrome's incognito mode? The cache might be a problem. |
@r0hit-gupta Note, this happen only in Android device |
Yes. It looks fine to me on OnePlus 6 - Android 8.1.0. Please try again once in incognito and if its still an issue, will look deeper into it. |
@r0hit-gupta Yeah, looks like was problem with caching. Thanks for checking. |
@luizrrodrigues sorry I was in travel. Just being curious, who has get paid? |
Hey @nahidshahin, payment already, check: |
Actual: Sub-menu should not display that extra item
Reproducibility Rate: 3/3
Environment: Pixel XL, Android 9
Browser: Chrome 75.0.3770.80
The text was updated successfully, but these errors were encountered: