Skip to content

Allow side pane to be completely hidden#3784

Merged
Siedlerchr merged 1 commit into
maintable-betafrom
fixSidebar
Feb 27, 2018
Merged

Allow side pane to be completely hidden#3784
Siedlerchr merged 1 commit into
maintable-betafrom
fixSidebar

Conversation

@tobiasdiez

Copy link
Copy Markdown
Member

Fixes points 3. and 4. of #3621 (comment)

Another issue is that the search-bar is connected to the splitter between main-table and the left side bar. When I resize the main-table, it is possible to move the search-bar over the toolbar icons:

When closing the sidebar, the main-table is not maximized anymore. As can be seen in the image above, we end up with a gray area where the side-bar was.


  • Change in CHANGELOG.md described
  • Tests created for changes
  • Manually tested changed features in running JabRef
  • Screenshots added in PR description (for bigger UI changes)
  • Ensured that the git commit message is a good one
  • Check documentation status (Issue created for outdated help page at help.jabref.org?)

@tobiasdiez tobiasdiez added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Feb 27, 2018
@tobiasdiez tobiasdiez mentioned this pull request Feb 27, 2018
35 tasks
@Siedlerchr Siedlerchr merged commit fc36140 into maintable-beta Feb 27, 2018
@Siedlerchr Siedlerchr deleted the fixSidebar branch February 27, 2018 15:39
Siedlerchr added a commit that referenced this pull request Feb 27, 2018
…gsjavafx

* upstream/maintable-beta:
  Save order of columns across sessions (#3783)
  Allow side pane to be completely hidden (#3784)
  Show empty group pane if no database is open (#3785)
  Reenable drag'n'drop support for tabs / libraries (#3688)
Siedlerchr added a commit that referenced this pull request Feb 28, 2018
…drop

* upstream/maintable-beta:
  Save order of columns across sessions (#3783)
  Allow side pane to be completely hidden (#3784)
  Show empty group pane if no database is open (#3785)
  Reenable drag'n'drop support for tabs / libraries (#3688)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants