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

Scrollbar shows up needlessly in group panel, while decreasing the width. #8467

Closed
2 tasks done
ThiloteE opened this issue Jan 26, 2022 · 9 comments · Fixed by #8756
Closed
2 tasks done

Scrollbar shows up needlessly in group panel, while decreasing the width. #8467

ThiloteE opened this issue Jan 26, 2022 · 9 comments · Fixed by #8756

Comments

@ThiloteE
Copy link
Member

JabRef version

5.5 (latest release)

Operating system

Windows

Details on version and operating system

JabRef 5.6--2022-01-25--2c92eb0 Windows 10 10.0 amd64 Java 16.0.2 JavaFX 17.0.1+1

Checked with the latest development build

  • I made a backup of my libraries before testing the latest development version.
  • I have tested the latest development version and the problem persists

Steps to reproduce the behaviour

  1. Expand the width of the group panel.
  2. Then decrease the width of the groups panel.

Result:

at certain widths, while decreasing the width, the row will show up.

Examples:

  • Dark Mode:
    image
  • Light Mode:
    image

Appendix

No response

@ThiloteE
Copy link
Member Author

Desired behaviour: This special row should show in the same colour as the rest of the background.

@ThiloteE
Copy link
Member Author

Ohh, it seems to be the scrollbar!

@ThiloteE ThiloteE changed the title Different colour row unintentionally shows up in group panel. Both in light and dark mode. Scrollbar shows up needlessly in group panel, while decreasing the width. Jan 26, 2022
@Siedlerchr
Copy link
Member

Yep, it's the scrollbar. Might be a javafx thing.. Not sure if this changable.

@ThiloteE
Copy link
Member Author

I will leave it open for the sake of acknowledging that it exists, but this issue is definitely low priority. More important issues around.

@LIM0000
Copy link
Contributor

LIM0000 commented May 4, 2022

Hi,
I'm interested in working on this issue as my first code contribution to open-source. Can I give it a shot?

@ThiloteE
Copy link
Member Author

ThiloteE commented May 4, 2022

This issue possibly already MIGHT have been solved in JFX 18.
See the release notes. There have been multiple fixes to scroll-bars. Following pull request will introduce JFX to JabRef: #8629

In case JFX 18 has NOT fixed this, work probably needs to be done here, instead of at JabRef. Going through the JFX issue list first is advised.

@ThiloteE
Copy link
Member Author

ThiloteE commented May 4, 2022

@LIM0000 this issue has low priority and it could be that it will be solved with jfx 18. Please check out issues with the label "good first issue". See here: https://github.com/JabRef/jabref/issues?q=is%3Aissue+is%3Aopen+label%3A%22good+first+issue%22.

@ThiloteE ThiloteE added the status: freeze Issues posponed to a (much) later future label May 4, 2022
@LIM0000
Copy link
Contributor

LIM0000 commented May 4, 2022

@ThiloteE Thank you for your quick response.

I have checked both the JFX issue lists and JFX 18 release notes, none of the notes have mentioned the ability to disable scroll bar in TreeTableView that is used to display columns of group panel for Jabref.

I have done some researches and found that TreeTableView has its own built-in scrollpane that causes this issue. By looking into Jabref codebase, this issue could be fixed by adding a style to TreeTableView horizontal scrollbar.

.tree-table-view .scroll-bar:horizontal {
    -fx-opacity: 0;
}

Hope this is helpful.

@ThiloteE
Copy link
Member Author

ThiloteE commented May 4, 2022

Well, since you already have done so much research and might have a possible fix, sure go ahead :) Open a pull-request and we will be able to see how it behaves.

As a general advice for newcomers: check out https://github.com/JabRef/jabref/blob/main/CONTRIBUTING.md for a start. Also, https://devdocs.jabref.org/getting-into-the-code/guidelines-for-setting-up-a-local-workspace is a good start. Feel free to ask if you have any questions here on GitHub or also at JabRef's Gitter chat.

Try to open a (draft) pull request early on, so that people can see you are working on the issue and so that they can see the direction the pull request is heading towards. This way, you will likely receive valuable feedback.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants