Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Clicking Bookmarks Toolbar from "Bookmarks" menu does not change state of check mark #4810

Closed
luixxiul opened this issue Oct 15, 2016 · 3 comments

Comments

@luixxiul
Copy link
Contributor

Describe the issue you encountered: Clicking Bookmarks Toolbar from "Bookmarks" menu does not change state of check mark. Even if you click it to hide the toolbar, the check mark still remains. This does not happen on the hamburger menu.

Expected behavior: It should change state of check mark.

  • Platform (Win7, 8, 10? macOS? Linux distro?): Windows 10
  • Brave Version: 0.12.5 RC1
  • Steps to reproduce:
    1. Clicking Bookmarks Toolbar from "Bookmarks" menu to show the toolbar
    2. Click it again to hide the toolbar -> the check mark remains
  • Any related issues:
@bbondy
Copy link
Member

bbondy commented Oct 15, 2016

This is working reliably for me on macOS (which I'd expect to be the same) but was a problem with an old build. Are you sure you had the latest?

@bbondy bbondy modified the milestones: 0.12.6dev, 0.12.5dev Oct 15, 2016
@bbondy
Copy link
Member

bbondy commented Oct 15, 2016

Tested on Windows and reproduced, I guess maybe the code is different because of the titlebar removal work

@bsclifton
Copy link
Member

Found root cause- menu status (the electron menu) is being updated, but not the version in the appState. I'll have a fix later tonite 😄

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.