PlasmaProject
ActivePublic

Recent Activity

Sat, Aug 8

concernedone created T13488: Report security, privacy, and software freedom issues via KDE software.
Sat, Aug 8, 1:55 PM · KDE Privacy Goal, KDE Applications, Plasma

Thu, Aug 6

clel added a comment to T12839: Fix sizing of first-party plasmoids in the panel.

I'd vote for that, can you create a mockup? Also, is the workspace switcher next to it having any margins applied? Mine is alot more simple, so I don't really know how that would look like. Currently for me it uses almost no margins (and I think I like it).

Thu, Aug 6, 10:18 AM · Plasma, VDG, Goal: Consistency
niccolove added a comment to T12839: Fix sizing of first-party plasmoids in the panel.

We can make the plasma icon ignore margins.

Thu, Aug 6, 9:43 AM · Plasma, VDG, Goal: Consistency
clel added a comment to T12839: Fix sizing of first-party plasmoids in the panel.
  1. a 44px panel with 22px widgets and 32px task manager (which would be consistent with applets who extend to panel margins, such as the pager)
Thu, Aug 6, 2:16 AM · Plasma, VDG, Goal: Consistency
Luwx added a comment to T12839: Fix sizing of first-party plasmoids in the panel.

Some feedback:

Thu, Aug 6, 12:40 AM · Plasma, VDG, Goal: Consistency

Wed, Aug 5

niccolove added a comment to T12839: Fix sizing of first-party plasmoids in the panel.

I have two proposals:

  1. a 44px panel with 22px widgets and 28px task manager

  1. a 44px panel with 22px widgets and 32px task manager (which would be consistent with applets who extend to panel margins, such as the pager)

Wed, Aug 5, 10:17 PM · Plasma, VDG, Goal: Consistency
clel added a comment to T12839: Fix sizing of first-party plasmoids in the panel.

For me unifying the size of the applets is not something we should strive for. The visual difference in the size of icons allows to communicate a visual hierarchy between elements, e.g. task manager icons are more often used than system trays icons.

Wed, Aug 5, 7:22 PM · Plasma, VDG, Goal: Consistency
ngraham added a comment to T12839: Fix sizing of first-party plasmoids in the panel.

I tested out your patches for that and I gotta say, I just think that those 22px Task Manager icons are too small. :( I have to agree with Carl and various other people who have voiced this opinion. Those icons look lost inside their highlight effect backgrounds.

Wed, Aug 5, 7:05 PM · Plasma, VDG, Goal: Consistency
niccolove added a comment to T12839: Fix sizing of first-party plasmoids in the panel.

After some discussion with Nate and Noah, these points were agreed upon:

  • upstream latte separator
  • add separator button
  • separator left and right of the tray
  • all widgets use 22px icons by default
  • 44px panel
  • systray stays 22px regardless of the panel size
  • option to make systray bigger as well
Wed, Aug 5, 3:17 PM · Plasma, VDG, Goal: Consistency
ognarb added a comment to T12839: Fix sizing of first-party plasmoids in the panel.

For me unifying the size of the applets is not something we should strive for. The visual difference in the size of icons allows to communicate a visual hierarchy between elements, e.g. task manager icons are more often used than system trays icons. Maybe as a good compromise, we should try to make the icons have the same height as the clock text, it is still bigger than the previous size but smaller than the current oversized icons and it will make the system trays icon size consistent with the elements near it, that is imho more important than with the rest of the panel.

Wed, Aug 5, 7:43 AM · Plasma, VDG, Goal: Consistency

Tue, Aug 4

ngraham added a comment to T12839: Fix sizing of first-party plasmoids in the panel.

Excellent. That looks great.

Tue, Aug 4, 7:41 PM · Plasma, VDG, Goal: Consistency
ngraham added a comment to D28651: Load and use global animation settings.

Agreed. @sandsmark, would you be interested in following up with that?

Tue, Aug 4, 7:40 PM · Plasma, Breeze
hpereiradacosta added a comment to D28651: Load and use global animation settings.

Thanks for this change !
Can the same be done in the breeze window decoration ? It is strange to have an animation settings there and not in the style.

Tue, Aug 4, 7:36 PM · Plasma, Breeze

Mon, Aug 3

niccolove added a comment to T12839: Fix sizing of first-party plasmoids in the panel.

Seems like your proposed approach already satisfies 1-3. We'll need to do 5, and maybe discuss whether we should do 4 as well.

One concern I have visually is that the Task Manager icons appear very small inside their now-a-bit-larger frames. They look a bit lost in there IMO. Not a huge deal though, and maybe it's just something to get used to.

Also I wonder how we're going to handle smaller panels? 8px margins are going to be much too large for skinny panels that I know a lot of people like. We'll want to reduce that progressively as panel size drops, probably eventually winding up with 2px margins for really thin panels.

Mon, Aug 3, 4:13 PM · Plasma, VDG, Goal: Consistency
ngraham added a comment to T12839: Fix sizing of first-party plasmoids in the panel.

Seems like your proposed approach already satisfies 1-3. We'll need to do 5, and maybe discuss whether we should do 4 as well.

Mon, Aug 3, 4:04 PM · Plasma, VDG, Goal: Consistency
ngraham added a comment to T12839: Fix sizing of first-party plasmoids in the panel.

All right. To me that suggests a relatively simple implementation:

  1. Make all widgets/plasmoids use the panel margins and do not add any internal margins of their own
  2. Increase the default panel margin
  3. Make the Task Manager ignore panel margins for its highlight effect, but respect them for icons.
  4. (optional) make the panel margin user-adjustible/overridable somewhere so that people who like bigger icons can get them without breaking size consistency. This could be exposed through a simplified UI expressed in the form of "Panel icons: smaller/bigger" or something like that.
  5. Make sure the Digital Clock widget still looks readable with bigger panel margins and our default panel size.
Mon, Aug 3, 3:58 PM · Plasma, VDG, Goal: Consistency
niccolove added a comment to T12839: Fix sizing of first-party plasmoids in the panel.

I think our goal is to provide a good balance of icon size consistency and panel aesthetic. The idea of making all icon of the same size is - I think - good, as long as those icons are not excessively big nor small, and I'm afraid that neither 32px nor 22px icon visually work on our default 46px panel when icons are all perfectly the same size. So we have to sacrifice a bit of consistency. With my idea (and, now, implementation) - that 100% looks like Manuel mockup - I think we get a good balance of both:

Mon, Aug 3, 3:54 PM · Plasma, VDG, Goal: Consistency
ngraham closed D28651: Load and use global animation settings.
Mon, Aug 3, 3:49 PM · Plasma, Breeze
ngraham accepted D28651: Load and use global animation settings.
Mon, Aug 3, 3:44 PM · Plasma, Breeze
ngraham added a comment to T12839: Fix sizing of first-party plasmoids in the panel.

I'd like to step back here and make sure we define the problem space adequately.

Mon, Aug 3, 2:53 PM · Plasma, VDG, Goal: Consistency
niccolove added a comment to T12839: Fix sizing of first-party plasmoids in the panel.

Regarding the technical implementation, I'd make Plasma able to selectively add margin or not for each widget separately; this way all widgets are supported. I wouldn't make the user able to toggle those in the UI yet, we could decide if it's appropriate to add that in the future based on user requests. Right now the plasmoid itself could decide whether to have margin (the default) or not. Task manager really *has* to ignore margin to extend to borders, but then the margins are added back through the task manager internal margins defined in the theme.

Mon, Aug 3, 9:54 AM · Plasma, VDG, Goal: Consistency

Sun, Aug 2

The-Feren-OS-Dev added a comment to T12839: Fix sizing of first-party plasmoids in the panel.

Here's my idea for how it could work:
Key:
ICON is a placeholder for the icon size in the applet. Also the same size as the general widget contents (icon, text, etc.) in the panel would be.
Silver ICON: Same as ICON, but for unhighlighted items to convey the icon size on those as well as on highlighted items
Dark Grey: Highlighted Widget/Item
White: Unhighlighted Widget/Item

Sun, Aug 2, 4:24 PM · Plasma, VDG, Goal: Consistency
niccolove added a comment to T12839: Fix sizing of first-party plasmoids in the panel.

The task manager ignores margin but we can make it apply internal ones as big as the panel ones, so the icon sizes should still be consistent; Manuel was proposing to downscale 64px app icons to a 22px space, which - considering 32px icon sizes - would be as big as the actual launcher icon:


So we would basically have consistency in the left side and right side, with the right side icons being slightly smaller; I think it's a good approach.

Sun, Aug 2, 4:10 PM · Plasma, VDG, Goal: Consistency
ngraham added a comment to T12839: Fix sizing of first-party plasmoids in the panel.

Yeah, though it's starting to feel arbitrary again:

Sun, Aug 2, 3:56 PM · Plasma, VDG, Goal: Consistency
niccolove added a comment to T12839: Fix sizing of first-party plasmoids in the panel.

The Application Launcher applet icon is okay, it's just that it looks too small next to the task manager. Imagine having this small icon in Manuel's mockup:


It would be visually unbalanced; it makes more sense to use a 32px here, I guess

Sun, Aug 2, 3:45 PM · Plasma, VDG, Goal: Consistency
ngraham added a comment to T12839: Fix sizing of first-party plasmoids in the panel.

That seems like a sensible approach overall. I agree that making is user-facing seems like overkill. As the lone large-system-tray-icon lover, I'm willing to give up my 32px icons on a 46px panel. :) If I have to go up to 50px or higher, that's all right.

Sun, Aug 2, 3:18 PM · Plasma, VDG, Goal: Consistency
niccolove added a comment to T12839: Fix sizing of first-party plasmoids in the panel.

It was also suggested to make this setting user-facing (by hovering the widget in panel edit mode); I wouldn't do that yet because this is something very technical that the average user shouldn't want to change sizes, plus it could result in inconsistent sizing in the icons that we'd instead like to be consistent (minimize all, tray, clock, etc). This doesn't mean that we couldn't add it in the future if it turns out to be more necessary than we thought.

Sun, Aug 2, 10:15 AM · Plasma, VDG, Goal: Consistency

Fri, Jul 31

mart added a comment to T10966: Unify how we refer to KRunner.

the thing that triggered my attention on it is the desktop context menu entry:

FYI, the systemsettings entry also changed, that's far more user facing.

Copying my comment from bugzilla:

"krunner" use is already released in 5.17 onwards.
Whilst I didn't necessarily like the change, another change again now means a new set of user interruptions and relearning which I'm even more against. We shouldn't flip-flop on things that affect the end user.

Fri, Jul 31, 5:22 PM · VDG, Plasma

Fri, Jul 24

davidedmundson added a comment to T10966: Unify how we refer to KRunner.

the thing that triggered my attention on it is the desktop context menu entry:

Fri, Jul 24, 1:21 PM · VDG, Plasma

Thu, Jul 23

apol closed T11945: kwin stylus as Resolved.

Should be working on Wayland as of Plasma 5.18 with improvements in 5.19.

Thu, Jul 23, 4:49 PM · Plasma
apol closed T11945: kwin stylus, a subtask of T11939: Plasma 5.18, as Resolved.
Thu, Jul 23, 4:49 PM · Plasma

Wed, Jul 22

ahiemstra added a comment to T10966: Unify how we refer to KRunner.

(well, even ksysguard doesn't have to remain named this ugly way, the rewrite could be an occasion to name it less horribly)

Wed, Jul 22, 11:04 AM · VDG, Plasma
mart added a comment to T10966: Unify how we refer to KRunner.

There are several arguments against it:

  • the thing that triggered my attention on it is the desktop context menu entry: usually context menus should have actions and description of what the action does as text
Wed, Jul 22, 10:41 AM · VDG, Plasma
jriddell added a comment to T10966: Unify how we refer to KRunner.

I think I agree with Nate here, it is a terrible K name and I've always avoided those but it's also an advanced users feature and those advanced users tend to be fine with K names. "KSysGuard" name should die though.

Wed, Jul 22, 10:08 AM · VDG, Plasma

Tue, Jul 21

ngraham added a comment to T10966: Unify how we refer to KRunner.

In my mind, "KRunner" is a brand name, like the name of an app. I see it as closer to "Dolphin" than for example "Purpose framework". My time in the Apple world exposed me to a similar feature called "Spotlight" which is used as a brand name and exposed to users with that name in the UI. KRunner is basically the same feature, so it seemed natural to similarly advertise it by its real name. I don't see this causing any problems for Apple in macOS.

Tue, Jul 21, 4:47 PM · VDG, Plasma

Mon, Jul 20

mart added a comment to T10966: Unify how we refer to KRunner.

https://bugs.kde.org/show_bug.cgi?id=424455

Mon, Jul 20, 10:05 AM · VDG, Plasma
mart reopened T10966: Unify how we refer to KRunner as "Open".

I would ask to reconsider this.
KRunner is a terrible jargon name, which is perfectly ok for a framework, but that's it.

Mon, Jul 20, 9:59 AM · VDG, Plasma
mart added a comment to D21341: Call KRunner KRunner.

This was a terrible decision.
KRunner is the worst jargon one can imagine and shouldn't be anywhere in user facing UI

Mon, Jul 20, 9:54 AM · Plasma

Fri, Jul 17

shubham abandoned D28686: [Battery Monitor] Choose better wording.

Pushed here 38ebcf09b1631175d5a1b4d857622c5313146a60

Fri, Jul 17, 10:38 AM · Plasma

Thu, Jul 16

D28686: [Battery Monitor] Choose better wording is now accepted and ready to land.

Go ahead and land this.

Thu, Jul 16, 6:16 PM · Plasma
ognarb accepted D28686: [Battery Monitor] Choose better wording.

Change makes sense to me, IIRC Kai doesn't have much time for KDE anymore

Thu, Jul 16, 5:35 PM · Plasma
shubham added a comment to D28686: [Battery Monitor] Choose better wording.

@broulik Ping?

Thu, Jul 16, 2:28 PM · Plasma

Wed, Jul 15

meven closed D26718: aggregate text for KRunner in DesktopView.

At moved to https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/148

Wed, Jul 15, 1:48 PM · Plasma

Mon, Jul 13

liushuyu abandoned D25539: feat(kcm): add revert timer.

This patch has been migrated to https://invent.kde.org/plasma/kscreen/-/merge_requests/1

Mon, Jul 13, 10:11 PM · Plasma
volkov abandoned D28683: Get rid of workaround that prevents file dialog from being blocked.

Moved to https://invent.kde.org/plasma/plasma-integration/-/merge_requests/1

Mon, Jul 13, 12:03 PM · Plasma

Sun, Jul 12

PhilipB added a comment to T8569: Redesign Policy Kit Authorization dialog.

Sun, Jul 12, 3:38 AM · VDG, Plasma
ngraham added a parent task for T8569: Redesign Policy Kit Authorization dialog: T12420: Things that could be redesigned or tweaked.
Sun, Jul 12, 1:38 AM · VDG, Plasma

Sat, Jul 11

fvogt added a comment to T2050: sunsetting KHotKeys.

In general I think it's sub-optimal to maintain two different tools that do essentially the same thing, even if both are working fine.

Sat, Jul 11, 9:39 AM · KF6, Plasma
markuss closed T10952: Drop EmojiOne and adopt Twemoji for emoticon substitution as Wontfix.

https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/52

Sat, Jul 11, 6:54 AM · Plasma

Jul 11 2020

ngraham added a comment to T2050: sunsetting KHotKeys.

In general I think it's sub-optimal to maintain two different tools that do essentially the same thing, even if both are working fine. It's confusing for users and it generates extra work for developers. If there are desirable things that KHotkeys can do which KGlobalAccel can't, IMO we should add the missing features to KGlobalAccel and continue with the plan to deprecate KHotKeys.

Jul 11 2020, 12:43 AM · KF6, Plasma