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

Vertically align Groups and Web search panel #8397

Closed
ThiloteE opened this issue Jan 7, 2022 · 27 comments · Fixed by #8463
Closed

Vertically align Groups and Web search panel #8397

ThiloteE opened this issue Jan 7, 2022 · 27 comments · Fixed by #8463

Comments

@ThiloteE
Copy link
Member

ThiloteE commented Jan 7, 2022

Problem:

Groups, the Add group button, Web search and the search button for Web search are centrally aligned and it looks odd, because the button "Toggle intersection" and the multiple buttons "Move panel up", "Move panel down" and "Hide panel" skew two of the four more towards the left.

Desired Solution:

Align the text for the four along a common line.

Example:

The text could start somewhere along the red line:
image

I would refrain from aligning it completely to the left border as there should be a contrast to groups and search fields.

@mlep
Copy link
Contributor

mlep commented Jan 7, 2022

@dimitrisdimos00 : since you have already touch some of this, maybe you would like to pursue?

@ThiloteE
Copy link
Member Author

ThiloteE commented Jan 7, 2022

Would something like this get the label UI or rather the label groups/search?

@mlep
Copy link
Contributor

mlep commented Jan 7, 2022

Both? 😉

@ThiloteE
Copy link
Member Author

ThiloteE commented Jan 7, 2022

ok xD

Btw. another alternative could be to align the buttons Add group and Search at the right side. There would still be contrast (the buttons would stand out more) and at the same time, the awkward looking centrally aligned indention vanishes.

I imagine both to be fine looking :)

@mlep
Copy link
Contributor

mlep commented Jan 7, 2022

I believe we are used to have the text of a button centered.
I suggest to wait for the merge of #8392 first, and then look at what can be improved further.

@dimitrisdimos00
Copy link
Contributor

Hello. Yeah, I can look into it. So, shall we wait for the merge and then take a look at it?

@ThiloteE
Copy link
Member Author

ThiloteE commented Jan 7, 2022

Yeah we can wait for the merger. This is not urgent as 5.4 stable was just released :)

@ilippert
Copy link
Contributor

I would imagine that it would also look well if the names "Groups" and "Web search" are aligned left.

@dimitrisdimos00
Copy link
Contributor

The first suggestion is to move the "Add group" button and the "Search button" to the left in order to be right below "Groups" and "Web search" accordingly. The second one is to move the "Groups" and "Web search" labels completely to the left and leave the buttons where they are. Did I get it right?

@mlep
Copy link
Contributor

mlep commented Jan 13, 2022

That is my understanding.
I guess we should consider all three panels together: Groups, OpenOffice/LibreOffice and Web search.

About the titles of the panel: they are a bit off-centered at the moment. Groups looks a bit displaced to the left at the moment when the panel is wide)

About their buttons:

  • "Add group" is centered.
  • "Search" is centered.
  • the OpenOffice/Libreoffice buttons (there are 10 of them) are grouped to the left.

ksnip_20220113-153206

@dimitrisdimos00
Copy link
Contributor

dimitrisdimos00 commented Jan 13, 2022

Yeah we definitely should look at all three of them. And we should decide which of the two we will implement. I don't have something in mind for the OpenOffice/LibreOffice buttons though.

@mlep
Copy link
Contributor

mlep commented Jan 18, 2022

A glitch that could be corrected in the meantime: while the Add group button extend to the whole panel width, the Search button of the web search panel does not. With the light theme, you need to put the pointer onto the button to see it, but in the dark theme it shows right away:
ksnip_20220118-090634

My suggestion is to make the Search button extend to the whole panel width.

@dimitrisdimos00
Copy link
Contributor

Yeah, I noticed that too... I can look into it

@dimitrisdimos00
Copy link
Contributor

Hey, check this out!
image
image

@dimitrisdimos00
Copy link
Contributor

So shall I try moving the titles to the left? I think it is more plausible

@ThiloteE
Copy link
Member Author

ThiloteE commented Jan 23, 2022

Yes, maybe you also can give us screenshots of the few options we have. We could vote, after all, there is more than one option how to deal with this :)

dimitrisdimos00 added a commit to dimitrisdimos00/jabref that referenced this issue Jan 24, 2022
@dimitrisdimos00
Copy link
Contributor

Here is how it looks when the titles are directed towards the left:
image

dimitrisdimos00 added a commit to dimitrisdimos00/jabref that referenced this issue Jan 24, 2022
@ThiloteE
Copy link
Member Author

How would this one look?

image

@dimitrisdimos00
Copy link
Contributor

I don't think I can align all the items completely because the buttons up, down, delete (x) and union (in the case of groups) push the titles (blue text) a little bit more to the left. All the items are centered already...

@ThiloteE
Copy link
Member Author

because the buttons up, down, delete (x) and union (in the case of groups) push the titles (blue text) a little bit more to the left.

Yes, this is the main reason, why i created this thread.

@Siedlerchr
Copy link
Member

Do not spend that much time on the alignment. It should be just easily distinguishable for the user from the rest of.

@ThiloteE
Copy link
Member Author

Oh well. Then let's give it #8397 (comment) a try.

@dimitrisdimos00
Copy link
Contributor

Ok I will open a pr then

dimitrisdimos00 added a commit to dimitrisdimos00/jabref that referenced this issue Jan 24, 2022
dimitrisdimos00 added a commit to dimitrisdimos00/jabref that referenced this issue Jan 24, 2022
dimitrisdimos00 added a commit to dimitrisdimos00/jabref that referenced this issue Jan 25, 2022
Siedlerchr pushed a commit that referenced this issue Jan 25, 2022
… to the left (#8463)

* Made Search button in Web Search panel as wide as the Add group button of the Groups side panel.
fixes #8397

* Skewed side pane title to the left.
fixes #8397

* Recorded change to CHANGELOG.md
fixes #8397

* Corrected type in CHANGELOG.md
fixes #8397

* Deleted forgotten comment
fixes #8397
@mlep
Copy link
Contributor

mlep commented Jan 26, 2022

@dimitrisdimos00 Thanks! Now, titles really look like titles (and buttons look like buttons).

Note: the titles are not vertically centered. They are slightly moved upward.
ksnip_20220126-173603

@dimitrisdimos00
Copy link
Contributor

dimitrisdimos00 commented Jan 26, 2022

Your welcome! Yeah about the vertical alignment, I don't know why that happens, I didn't notice it.

@ThiloteE
Copy link
Member Author

Yeah, looking good :)
I especially like how the titles look in dark mode.

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