Skip to content
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

add View -> Show Statusbar check menu item, plus key binding to show/hide statusbar #986

Closed
wants to merge 5 commits into from

Conversation

ilius
Copy link

@ilius ilius commented Apr 5, 2016

Also adding update_menu arg to ui_statusbar_showhide function, to fix "Toggle All Additional Widgets" problem and cleaner code

We should do the same for Sidebar too (adding gboolean update_menu to ui_sidebar_show_hide function), because when you click "Toggle All Additional Widgets" , the menu item (View -> Show Sidebar) is not checked

@elextr
Copy link
Member

elextr commented Apr 5, 2016

Please update the manual.

@codebrainz
Copy link
Member

@ilius FYI, you can update existing PRs without creating new one. Just push more commits to your feature branch and the PR will be updated.

Also, I think this part in the manual might be what @elextr referring to.

@ilius
Copy link
Author

ilius commented Apr 6, 2016

Thanks for the info, I didn't notice that
Next time I will create a new branch instead of adding PR from master

I just fixed another small bug
It's OK now

…hide statusbar

add update_menu arg to ui_statusbar_showhide function to fix "Toggle All Additional Widgets" problem and cleaner code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants