Bug 563585 - Tabs in the tab stack have no separators
Summary: Tabs in the tab stack have no separators
Status: NEW
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.16   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Platform-UI-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2020-05-26 08:38 EDT by Andrey Loskutov CLA
Modified: 2020-10-05 15:27 EDT (History)
4 users (show)

See Also:


Attachments
new theme vs default system L&F on Linux (without theme) (270.90 KB, image/png)
2020-05-26 08:38 EDT, Andrey Loskutov CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Andrey Loskutov CLA 2020-05-26 08:38:11 EDT
Created attachment 283019 [details]
new theme vs default system L&F on Linux (without theme)

Follow up on bug 538740. I usually don't use themes, but just tried how it looks like with the new squared tabs.

I see one major regression: the tabs have no separators drawn between them, this looks not nice. See attached screen shot with the default new UI L&F compared with "no themes" on Linux.
Comment 1 Lars Vogel CLA 2020-05-26 08:47:24 EDT
Thanks Andrey, looks like nobody noticed this in the M3 validation week.

Mike, Andrew or Pierre-Yves, can you have a look?
Comment 2 Andrew Obuchowicz CLA 2020-05-26 08:55:20 EDT
I personally don't mind the lack of seperator's, but VSCode & Firefox do use tab seperators so we probably should do the same.

I can most-likely contribute a patch, although Pierre-Yves or Mike might be able to do it faster (as improving the Maven language server is a top priority for me until 4.16 is released).
Comment 3 Mike Marchand CLA 2020-05-26 09:48:01 EDT
Hey Lars, I think the reason it was never noticed is because we've never had separators for tabs in the CTabRendering.  I can try to have a look once I solve a regression that was introduced with the introduction of square tabs.
Comment 4 Lars Vogel CLA 2020-06-03 05:27:36 EDT
Mike, can you take this? I also think separators are nicer.
Comment 5 Andrew Obuchowicz CLA 2020-06-03 09:40:13 EDT
Mike, when you create a patch please add me as a reviewer :)

And if you get caught up, I can try making a patch as well (once 4.16 is released) just let me know.
Comment 6 Lars Vogel CLA 2020-06-16 08:28:16 EDT
(In reply to Andrew Obuchowicz from comment #5)
> Mike, when you create a patch please add me as a reviewer :)
> 
> And if you get caught up, I can try making a patch as well (once 4.16 is
> released) just let me know.

+1
Comment 7 Lars Vogel CLA 2020-06-17 11:15:15 EDT
Mike, are you still planning to work on this?
Comment 8 Mike Marchand CLA 2020-06-17 11:42:54 EDT
I can try to take a crack at this over the next week or two.
Comment 9 Lars Vogel CLA 2020-07-20 07:19:05 EDT
(In reply to Mike Marchand from comment #8)
> I can try to take a crack at this over the next week or two.

Any update?
Comment 10 Mike Marchand CLA 2020-07-20 08:39:49 EDT
I will use some time this week to see what it'll take.
Comment 11 Andrey Loskutov CLA 2020-08-19 06:43:47 EDT
(In reply to Mike Marchand from comment #10)
> I will use some time this week to see what it'll take.

Any chance to have the patch for 4.17?
Comment 12 Mike Marchand CLA 2020-08-19 07:04:51 EDT
That was my goal.  Unfortunately, with a loss in the family recently, I have not put the time into this that I had initially planned.

I wanted to solve this in a way that the separators were optional, that way we could maintain the current look by tuning some CSS.