In T10243#236012, @tosta wrote:

Another icon that need improvement is elisa.svg. It looks bad with dark background.

Now that Kate has its own special icon, I don't think Kwrite needs to be changed.

sounds good

What if we also had another suffix for requesting color icons when a widget style defaults to using symbolic icons? I think that unless we have a way to specifically ask for color, we might run into different problems.

Sounds like a good idea.

Marco said he wasn't actually against it.

In T10891#234131, @baran wrote:

I am not sure if this is the correct place to ask this question. Sorry, if it is not. This is unrelated to the theme.

I am foreign to KDE and Plasma but I noticed that in the Plasma settings mockup, the title "Icons" is separate from the content. Why this is the case? Wouldn't it be better if the title and the settings were somehow more together. For instance, the title would be centered too. I don't know but if every other program uses left aligned titles, maybe, this left aligned way is more consistent.

Also, do we need to implement this separately for QML apps, Qt Widgets apps and Plasmashell?

:/ I was about to say how glad I was to hear that we have a clear path forward. @mart What didn't you like about it and what would you suggest doing instead?

alright

In T12789#233508, @ngraham wrote:

When you have a list of items where some have a subtitle and some do not, maybe all of the list items should have the same height anyway. Lists of items with different heights is really weird looking.

But probably all of them should have a subtitle, or none of them should.

Giving this a +1 just for consistency with the kirigami and plasma-framework changes. I can't really test this since I don't have any screens with different DPIs though.

In T12789#233026, @nicolasfella wrote:

I like the idea of having a larger icon when there is a subtitle. However when trying to do it for all items with more than one line it leads to a somewhat awkward look with mixed line heights

• For 2 lines of text with color icons, use 32px.
• For 1 line of text with monochrome icons, use 22px.
• For 1 line of text with a logo, maybe use 32px so that the logo is easier to see?

@churaev Please don't derail the discussion.

I think I agree with the general idea, but it's a bit hard to fully grasp the scope of what must be done and I'm not sure what our first steps should be.

I think I'd prefer to keep this similar to the original Kirigami icon. Do you mind if I commandeer this? I've already made the changes.

I noticed that the lighting in this version of the Kirigami is flipped. Is that intentional? I don't think it makes sense for the top of the K popout to be darker than the bottom. Since the light in Breeze icons comes from the top left, the top should be light, like in the original Kirigami icon.

will move to invent.kde.org

will move to invent.kde.org

How would we add the separator role to color scheme files if separator color was added to upstream Qt? Wouldn't we still need to add a separator role to KColorScheme so that we could map the color in the color scheme file to the equivalent QPalette ColorRole?

In D29477#673166, @meven wrote:

In D29477#673039, @ndavis wrote:

Resuming this on invent

link ?

Suggested colors are based on https://invent.kde.org/plasma/breeze/-/merge_requests/2

Is there a way to set the default panel based on info about the user's screen?

The line seems to stop just short of the sidebar:

what are some ways the issue of toolbar colors not being perfectly in sync with titlebar colors could be fixed or worked around?

It might be best not to change anything about the position until we can come up with a significantly better solution.

In D29263#673056, @ngraham wrote:

In the absence of a way for the Global Theme or (proposed-but-not-yet-existing OSD themes) to set a custom OSD position, are KWin people okay with this patch to improve the positioning of the new OSD?

Saving this for later on invent.

Resuming this on invent

Abandoning this since others would rather have this in QPalette

Resuming this on invent

Abandoning this since nobody wanted it.

Resuming this on invent