Pitel (Radek Hušek)
User

Projects

User does not belong to any projects.

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Wednesday

  • Clear sailing ahead.

User Details

User Since
Feb 16 2018, 8:50 PM (31 w, 2 d)
Availability
Available

Recent Activity

May 14 2018

Pitel added a comment to D11071: [RFC] [Active Window Control] Allow global menu be left alligned with buttons on right.

@martinkostolny I will try to give it some testing this week.

May 14 2018, 6:07 PM · Plasma

Apr 3 2018

Pitel added a comment to D11746: Assign notification area category.

@wsdfhjxc can you land it yourself or should I do it? I would like to land this, D11748, D11749, D11750, and D11751, so D11352 can land without the category override hack.

Apr 3 2018, 6:31 PM · Plasma
Pitel committed R120:bd5619e0155a: [Media controller] Add simple volume control (authored by Pitel).
[Media controller] Add simple volume control
Apr 3 2018, 6:25 PM
Pitel closed D11021: [Media controller] Add simple volume control.
Apr 3 2018, 6:25 PM · Plasma
Pitel added a comment to D11071: [RFC] [Active Window Control] Allow global menu be left alligned with buttons on right.

show on: mouse out | mouse in | always (and possibly never show unless it is covered by other options, show on mouse out not available for buttons and appmenu)

This one I don't fully understand. Why show on: mouse_out|mouse_in for all items is not enough to satisfy all scenarios?

They are if you think about them as two checkboxes, I thought about it as single combobox...

Apr 3 2018, 6:43 AM · Plasma

Mar 31 2018

Pitel added a comment to D11843: Show the scrollbar only when needed.

I know the stripe is still useless but user sees "it is disabled scrollbar, ok", not "empty black stripe, what is going on".

Mar 31 2018, 9:03 PM · Konsole
Pitel added a comment to D11843: Show the scrollbar only when needed.

Thanks for the screenshot. I think this will raise questions "why is there the ugly black stripe on right"... (But I generally dislike all overlay and overaly-like-looking scrollbars on desktop, so I am biased.)

Mar 31 2018, 8:45 PM · Konsole
Pitel added a comment to D11843: Show the scrollbar only when needed.

How does it work if text fills the whole line (or if you run mc or some other ncurses application)? Does it wrap the text once scrollbar is shown, hide it or is the text always wrapped in front of scrollbar's rect even if the scrollbar is not visible?

Mar 31 2018, 8:37 PM · Konsole
Pitel added a watcher for Plasma: Pitel.
Mar 31 2018, 6:24 PM
Pitel added a watcher for KDE Connect: Pitel.
Mar 31 2018, 6:23 PM
Pitel updated the diff for D11410: [RFC] Add option to make applet fixed width.

Fix tooltip text.

Mar 31 2018, 1:53 PM · Plasma

Mar 30 2018

Pitel added inline comments to D11410: [RFC] Add option to make applet fixed width.
Mar 30 2018, 7:33 PM · Plasma
Pitel updated the diff for D11021: [Media controller] Add simple volume control.

Convert changeVolume helper into a Job... Right now it does not check wheter the DBus call is successful - I tried to wire it in but ended with segfault. I will investigate it further. Except that I hope it is ok.

Mar 30 2018, 7:06 PM · Plasma
Pitel added a comment to D11262: KDE logout screen background color fix.

I am not sure what artifacts is he referring to (but I am usually blind to this kind of issues)...

I'm not.

I did not mean that I want to ignore them, I just need them pointed out for me to start seeing them.

Mar 30 2018, 5:45 PM · Plasma
Pitel added a comment to D11262: KDE logout screen background color fix.

It also should be in Plasma/5.12 now.

Mar 30 2018, 5:22 PM · Plasma
Pitel committed R120:ba749fad105a: Merge branch 'Plasma/5.12' (authored by Pitel).
Merge branch 'Plasma/5.12'
Mar 30 2018, 5:20 PM
Pitel committed R120:fc0033ede6b0: KDE logout screen background color fix (authored by Pitel).
KDE logout screen background color fix
Mar 30 2018, 5:20 PM
Pitel added a comment to D11262: KDE logout screen background color fix.

@ragreen I am sorry to disappoint but this patch does not remove the (usually black) half-opaque overlay the G+ post refers to. It only changes its color to button background color in the case that button background color is too far from black to make sure the text is readable. I guess the readability of text was the reason to introduce the overlay in the first place because only blurring the background (like 5.8 seems to do) is not enough in general.

Mar 30 2018, 5:16 PM · Plasma
Pitel added a comment to D11262: KDE logout screen background color fix.

It is meant as one. I am still kinda new here, so I am not exactly sure how to that (without confusing Phabricator).

Mar 30 2018, 2:30 PM · Plasma
Pitel added a comment to D11262: KDE logout screen background color fix.

I guess I should have renamed it from isLightColor to something like isNotAlmostBlack... The point was to replace almost black colors with real black because it looks better with opacity. IMHO (0,0,128), which is deep blue, is far enough from black to not mess with it because font colors readable on deep blue background might not be readable on black one.

Mar 30 2018, 2:14 PM · Plasma
Pitel committed R120:af2a74341d5f: KDE logout screen background color fix (authored by Pitel).
KDE logout screen background color fix
Mar 30 2018, 1:25 PM
Pitel closed D11262: KDE logout screen background color fix.
Mar 30 2018, 1:25 PM · Plasma

Mar 29 2018

Pitel updated the diff for D11352: [RFC] Auto ordered systray.
  • Remove debug print
Mar 29 2018, 4:30 PM · Plasma
Pitel added inline comments to D11352: [RFC] Auto ordered systray.
Mar 29 2018, 4:27 PM · Plasma
Pitel committed R884:f48d5afac734: Show window title when app name is missing (authored by Pitel).
Show window title when app name is missing
Mar 29 2018, 7:48 AM
Pitel closed D11072: [Active Window Control] Show window title when app name is missing.
Mar 29 2018, 7:48 AM · Plasma
Pitel committed R884:1119c47f1756: [Active Window Control] Slightly fix vertical alignment of window name (authored by Pitel).
[Active Window Control] Slightly fix vertical alignment of window name
Mar 29 2018, 7:38 AM
Pitel closed D11065: [Active Window Control] Slightly fix vertical alignment of window name.
Mar 29 2018, 7:38 AM · Plasma

Mar 27 2018

Pitel added a comment to D11262: KDE logout screen background color fix.

I finally got to do some testing and it is not looking good for luminance formula 0.2126*r + 0.7152*g + 0.0722*b.

Mar 27 2018, 3:56 PM · Plasma
Pitel updated the diff for D11410: [RFC] Add option to make applet fixed width.
  • Remove QtQuick.Controls import (was needed only by Checkbox)
Mar 27 2018, 2:53 PM · Plasma

Mar 25 2018

Pitel updated the diff for D11410: [RFC] Add option to make applet fixed width.
  • Replace checkbox with a checkable button
  • Add support for panel spacer -- currently we just check whether the applet is panel spacer and if it is we use applet.configuration.expanding instead of our fixedSizeOverride. I think we should remove context action to set expanding from panel spacer after this is landed and then convert it to use fixedSizeOverride array.
Mar 25 2018, 8:09 PM · Plasma
Pitel added a comment to D11352: [RFC] Auto ordered systray.

i would prefer the final version to be quiet

Sure.

Mar 25 2018, 7:09 PM · Plasma

Mar 21 2018

Pitel abandoned D11292: [RFC] Ordered system tray v2.

Replaced by D11352.

Mar 21 2018, 12:41 PM · Plasma
Pitel updated the diff for D11352: [RFC] Auto ordered systray.

Great, but I found one more bug (and it affected order of items). This fixes it.

Mar 21 2018, 11:14 AM · Plasma

Mar 20 2018

Pitel added a comment to D11352: [RFC] Auto ordered systray.

I am using Czech translation and I am satisfied with item order (but I keep only a few items in my systray). I like the current rule because it is simple and have some internal logic (even though it might not be obvious what the text of an applet is) and it is not just a fixed random order. I would rather not implement some crazy compare function just because it has somewhat reasonable results.

Mar 20 2018, 5:12 PM · Plasma
Pitel added inline comments to D11021: [Media controller] Add simple volume control.
Mar 20 2018, 1:42 PM · Plasma
Pitel added reviewers for D11352: [RFC] Auto ordered systray: VDG, Plasma.
Mar 20 2018, 11:31 AM · Plasma
Pitel added a comment to D11021: [Media controller] Add simple volume control.

ping

Mar 20 2018, 11:10 AM · Plasma
Pitel updated the diff for D11352: [RFC] Auto ordered systray.

ty, updated

Mar 20 2018, 8:32 AM · Plasma
Pitel updated the diff for D11352: [RFC] Auto ordered systray.
  • put UnknownCategory first
  • treat applets with category not in the list as UnknownCategory
  • split reorderItem function into two
  • add temporary hack whcih allows overriding applet's category (applets with wrong category are from diferent repos, so till they are fixed I added this hack, also does not contain Keyboard and Discover applets because I was unable to find their itemIds)
Mar 20 2018, 8:08 AM · Plasma

Mar 19 2018

Pitel updated the diff for D11410: [RFC] Add option to make applet fixed width.
In D11410#229277, @mart wrote:

it's moving on the right track.. i would like the checkbox to appear like the one in the panel spacer, in the context menu could be tricky as would need to inject into the applet's contextualactons, so let's forget about it for now.

I'm not sure that is good idea: There are already some items injected in context menus (e.g. Unlock widgets action) and looking at my panel some applets' context menus are not injected. I believe it is because they have multiple menus bound to specific parts of itself and only applet's global menu is (and can) be injected. Most notable examples are systray except the expansion arrow and task icon manager except empty space (if it has set fixedWidth I was unable to find a place in it which would show me context menu containing Unlock widget action).

Mar 19 2018, 7:04 PM · Plasma

Mar 17 2018

Pitel added a reviewer for D11410: [RFC] Add option to make applet fixed width: Plasma.
Mar 17 2018, 12:30 PM · Plasma
Pitel added a comment to D11410: [RFC] Add option to make applet fixed width.

Also do we want this or two options instead:

  • no expanding: maximumWidth = preferredWidth, and
  • no shrinking: minimumWidth = preferredWidth?
Mar 17 2018, 12:30 PM · Plasma
Pitel abandoned D10944: [taskmanager applet] Do not fill width.

Replaced by D11410.

Mar 17 2018, 10:35 AM · Plasma
Pitel requested review of D11410: [RFC] Add option to make applet fixed width.
Mar 17 2018, 10:33 AM · Plasma

Mar 16 2018

Pitel updated the diff for D11352: [RFC] Auto ordered systray.

@mart you were very right to point out the callLater because that is exactly what was broken. The code assumed that all calls of callLater with different arguments are executed but in fact only those with different function are, rest of arguments does not matter. I somehow managed to run an older version of the patch on my machine were callLater was applied to a local helper function so it worked for me...

Mar 16 2018, 5:04 PM · Plasma
Pitel planned changes to D11352: [RFC] Auto ordered systray.

Ok, that looks that I did screw something. Time to investigate...

Mar 16 2018, 4:24 PM · Plasma
Pitel added a comment to D11352: [RFC] Auto ordered systray.

Phabricator:Pitel 2:0

Mar 16 2018, 11:14 AM · Plasma
Pitel added a comment to D11352: [RFC] Auto ordered systray.

I guess Phabricator does not let me only respond to inline comments.

Mar 16 2018, 11:09 AM · Plasma
Pitel added a comment to D11352: [RFC] Auto ordered systray.

Doesn't seem to work for me. Only Notifications item is visible by default while there are multiple items set to be shown in the configuration. Also, changing the visibility state doesn't make any difference and the items are neither in panel nor in hidden panel, they just disappear. Only after going through configuration and manually switching every item to hidden and then to shown makes most of them visible.

Mar 16 2018, 8:04 AM · Plasma

Mar 15 2018

Pitel added a comment to D11292: [RFC] Ordered system tray v2.

To compare with auto ordering (as @mart would like) see D11352.

Mar 15 2018, 12:48 PM · Plasma
Pitel requested review of D11352: [RFC] Auto ordered systray.
Mar 15 2018, 12:40 PM · Plasma
Pitel added a comment to D11072: [Active Window Control] Show window title when app name is missing.

I do not have write access.

Mar 15 2018, 5:28 AM · Plasma
Pitel added a comment to D10837: bad textarea width for rename file if filename too short.

@painlessroaster is not me. What gave you impression he is?

Mar 15 2018, 5:27 AM · Plasma

Mar 14 2018

Pitel added a comment to D11308: Use the default Plasma wallpaper on the lock screen.

+1 for picture instead of blue. With the current UI I would really appreciate option to choose text color independently on plasma color scheme (and such option would solve any static picture). Other possibility I was thinking about was 50 % opaque (default black) rectangle (with rounded corners) around control elements (i.e. like logout screen but only around controls not whole screen) but I have not found the way to position it where I want it to be...

Mar 14 2018, 4:38 PM · Plasma
Pitel added a comment to D10944: [taskmanager applet] Do not fill width.

Ahh sorry I misunderstood you. Moving this option to containment level seems reasonable although I'm not sure for how many other applets it might be useful -- except taskbar and Active Windows Control (the global menu applet in my screen shot and it already has this option) all applets I use are fixed width.

Mar 14 2018, 12:27 PM · Plasma
Pitel added a comment to D10944: [taskmanager applet] Do not fill width.

I understand you do not like adding options that you do not like adding options adjust think which should be configured on another level. But to be able to configure filling empty space at level of panel applets must not claim they need more space than they really do (and now taskmanager is doing exactly that). So possible solutions I see are:

Mar 14 2018, 11:42 AM · Plasma
Pitel updated the diff for D11292: [RFC] Ordered system tray v2.
  • modified onConfigurationChanged so it triggers updates of properties depending on cfg_shownItems and cfg_hiddenItems
  • make trash bin not draggable
  • removed ComboBox hack (which was not working anyway) and added an overlay label to combobox showing S for items in cfg_shownItems and H for items in cfg_hiddenItems (this is meant as debugging tool only) -- when dragging the visibility values in comboboxes go wrong but those letter remain correct which leads me to conclusion that the problem is ComboBox component (@wsdfhjxc can you verify this behavior?)
Mar 14 2018, 6:46 AM · Plasma

Mar 13 2018

Pitel added a comment to D10944: [taskmanager applet] Do not fill width.

Ping. The current version adds a configuration option fillWidth (enabled by default) which allows the applet to grow beyond its preferred size. I tested it in usual setup and with fillWidth enabled the behavior is unchanged.

Mar 13 2018, 9:15 PM · Plasma
Pitel added a comment to D11292: [RFC] Ordered system tray v2.

FWIW, the crash you're referring to is also fixed with an up-to-date (not released yet) Qt.

Mar 13 2018, 9:05 PM · Plasma
Pitel added a comment to D11292: [RFC] Ordered system tray v2.

I am aware of getFixedItemId, only forgot to mention it in summary. Right now it is not included mostly because it is another hack (but fixing stupid behavior of someone else so unavoidable) and I do not use DropBox, but if this gets any closer to landing, we should reintroduce it.

Mar 13 2018, 7:01 PM · Plasma
Pitel updated the summary of D11292: [RFC] Ordered system tray v2.
Mar 13 2018, 6:18 PM · Plasma
Pitel added a comment to D11233: Ordered system tray.

I really like the idea of being able to reorder the systray as I wish. I could not resist and took my stab on the issue (with drag&drop) D11292.

Mar 13 2018, 5:36 PM · Plasma
Pitel updated the summary of D11292: [RFC] Ordered system tray v2.
Mar 13 2018, 5:34 PM · Plasma
Pitel requested review of D11292: [RFC] Ordered system tray v2.
Mar 13 2018, 5:31 PM · Plasma
Pitel added a comment to D11262: KDE logout screen background color fix.

I see that relative luminance formula is more correct but I would like to point out that we do not need here to calculate precise luminance. The point of the ?: statement is to collapse colors close to back to real black because gray overlay with 50 % opacity does not look good. So the current formula looks good enough for me and I would even think about moving the threshold down a bit to 0.4 or 0.3. (Generally not collapsing to black is the save way, it may not look so good but the text will be still easy to read.)

Mar 13 2018, 4:58 PM · Plasma

Mar 12 2018

Pitel added a comment to D11072: [Active Window Control] Show window title when app name is missing.

Do I need to do anything to make it land?

Mar 12 2018, 8:59 PM · Plasma
Pitel added a comment to D11262: KDE logout screen background color fix.

Would you be interested in producing similar patches for the Lock and Login screens, which suffer from the same issue?

Mar 12 2018, 5:25 PM · Plasma
Pitel added a comment to D11262: KDE logout screen background color fix.

With dark button background as it used to


but if you choose some crazy combination (like black text on green background) it will also work (without this patch the background here would be still black)

Mar 12 2018, 2:25 PM · Plasma
Pitel added a comment to D7537: KDE logout screen - change black icons & texts to white/greyish.

I finally found what was wrong here: the background should not be black when button background color is light but the calculation introduced by D5036 is wrong. My attempt to fix it is D11262.

Mar 12 2018, 12:50 PM · Plasma
Pitel requested review of D11262: KDE logout screen background color fix.
Mar 12 2018, 12:48 PM · Plasma

Mar 11 2018

Pitel added a comment to D7537: KDE logout screen - change black icons & texts to white/greyish.

I guess I am not that confused anymore -- with normal color group the color of text (outside of buttons) is determined by text color theme setting but with complementary group by button text color.

Mar 11 2018, 9:58 AM · Plasma
Pitel added a comment to D7537: KDE logout screen - change black icons & texts to white/greyish.

@mvourlakos I did some more testing and I am really confused. The important color seems to be button text color in Plasma theme setting. Results:

Mar 11 2018, 9:01 AM · Plasma
Pitel added a comment to D7537: KDE logout screen - change black icons & texts to white/greyish.

I have the same problem in 5.12.2 release. I think the root of it is assuming that foreground in PlasmaCore.Theme.ComplementaryColorGroup is white(ish) and background is dark (this is true only if normal color scheme uses dark text on light background, which is default) and forcing that background is really dark (since D5036). My personal hack is just to replace ComplementaryColorGroup with NormalColorGroup (which obviously breaks default setup). A bit less of a hack would be choosing either NormalColorGroup or ComplementaryColorGroup depending on which one has darker background.

Mar 11 2018, 6:55 AM · Plasma

Mar 7 2018

Pitel updated the diff for D11065: [Active Window Control] Slightly fix vertical alignment of window name.

Moving appmenu instead of window title.

Mar 7 2018, 8:02 AM · Plasma
Pitel added a comment to D11071: [RFC] [Active Window Control] Allow global menu be left alligned with buttons on right.

The patch definitely can wait until you finish the rework. A few thoughts that crossed my mind when reading proposed changes:

Mar 7 2018, 7:47 AM · Plasma

Mar 5 2018

Pitel requested review of D11073: [RFC] [Active Window Control] Elide appmenu when it is too long.
Mar 5 2018, 9:11 PM · Plasma
Pitel requested review of D11072: [Active Window Control] Show window title when app name is missing.
Mar 5 2018, 8:54 PM · Plasma
Pitel requested review of D11071: [RFC] [Active Window Control] Allow global menu be left alligned with buttons on right.
Mar 5 2018, 8:45 PM · Plasma
Pitel added a comment to D11065: [Active Window Control] Slightly fix vertical alignment of window name.

Would it be better to move global menu 1px down instead?

Mar 5 2018, 7:13 PM · Plasma
Pitel requested review of D11065: [Active Window Control] Slightly fix vertical alignment of window name.
Mar 5 2018, 6:37 PM · Plasma
Pitel updated the diff for D11021: [Media controller] Add simple volume control.

The changeVolume helper must be in PlayerControl class and MultiplexedService must only forward calls to it in order to volume change by mouse wheel also work for other sources than only mutliplex one.

Mar 5 2018, 3:59 PM · Plasma
Pitel updated the summary of D11021: [Media controller] Add simple volume control.
Mar 5 2018, 12:55 PM · Plasma
Pitel added a reviewer for D11021: [Media controller] Add simple volume control: broulik.
Mar 5 2018, 12:53 PM · Plasma
Pitel updated the diff for D11021: [Media controller] Add simple volume control.
  • Volume is bounded at max(oldVolume, 1.0).
  • Added OSD support.
  • Refactored osd & volume bounding logic into helper MultiplexedService::changeVolume.
Mar 5 2018, 12:44 PM · Plasma
Pitel updated the summary of D11021: [Media controller] Add simple volume control.
Mar 5 2018, 12:20 PM · Plasma

Mar 4 2018

Pitel retitled D11021: [Media controller] Add simple volume control from [Media contoller] Add simple volume control to [Media controller] Add simple volume control.
Mar 4 2018, 8:49 PM · Plasma
Pitel added a comment to D11021: [Media controller] Add simple volume control.

The idea crossed my mind but I rarely use expanded version of the applet and there are two problems to deal with:

Mar 4 2018, 4:15 PM · Plasma
Pitel requested review of D11021: [Media controller] Add simple volume control.
Mar 4 2018, 3:49 PM · Plasma

Mar 2 2018

Pitel updated the diff for D10944: [taskmanager applet] Do not fill width.

In previous revisions I used fillWidth = false but it is wrong because it not only prevents expanding but also it prevents applet from shrinking. So use Layout.maximumWidth = Layout.preferredWidth instead. Also after further testing I do not think the option must be restricted to icon version of applet only -- with full version (and Spacer applet) it allows you to align task on right side of panel which was impossible before.

Mar 2 2018, 2:09 PM · Plasma
Pitel added a comment to D10944: [taskmanager applet] Do not fill width.

Screenshots with Global Menu applet (panel contains Global menu, growing Spacer and icon taskmanager).

Mar 2 2018, 10:35 AM · Plasma

Mar 1 2018

Pitel added a comment to D10944: [taskmanager applet] Do not fill width.

Not really -- if I understand correctly allocating space works following way: All widgets get space they claim they need and the surplus is split among widgets with fillWidth == true in ratio which seems proportinal to width of given widgets. So even if global menu took space needed to display all menu items there still would be an ugly gap on right side of taskmanager.

Mar 1 2018, 5:45 PM · Plasma
Pitel updated the diff for D10944: [taskmanager applet] Do not fill width.

Choice whether to fill or not fill width was turned into config option and i possible to set (and used) only for iconsOnly
version of taskmanager.

Mar 1 2018, 5:06 PM · Plasma
Pitel updated the summary of D10944: [taskmanager applet] Do not fill width.
Mar 1 2018, 2:58 PM · Plasma
Pitel requested review of D10944: [taskmanager applet] Do not fill width.
Mar 1 2018, 2:46 PM · Plasma