D29823: Increasing the zoom effect and adding "launcherBounceLevel" config option

mvourlakos added a comment. 1. Provide videos with before and after 2. I dont think I am interested in adding a bounce slider in Tasks for this 3. You increase the max zoom factor to 500% this is out of scope 4. I can accept an improvement for bouncing animation but without [2] and [3]

D29823: Increasing the zoom effect and adding "launcherBounceLevel" config option

mvourlakos added a comment. if you want to increase the bouncing level the right line is at: https://invent.kde.org/plasma/latte-dock/-/blob/master/plasmoid/package/contents/ui/main.qml#L982 you just need to increase the value of 1.65 to something bigger. Otherwise the calculated

D29823: Increasing the zoom effect and adding "launcherBounceLevel" config option

parsa created this revision. parsa added a project: Latte Dock. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. parsa requested review of this revision. REVISION SUMMARY I want to make the parabolic zoom effect more customizable.And in order to achieve that I had to

D29823: Increasing the zoom effect and adding "launcherBounceLevel" config option

parsa added a comment. @mvourlakos Yes this is subjective .But it would be left as an option for users who are after a more customizable dock. REPOSITORY R878 Latte Dock REVISION DETAIL https://phabricator.kde.org/D29823 To: parsa, mvourlakos Cc: plasma-devel, acemond, Orage,

D29823: Increasing the zoom effect and adding "launcherBounceLevel" config option

mvourlakos requested changes to this revision. mvourlakos added a comment. This revision now requires changes to proceed. 5. this patch only increases the thickness of launcher animation I dont consider this an improvement. This is subjective REPOSITORY R878 Latte Dock REVISION DETAIL

D28154: Add users KCM

cblack updated this revision to Diff 83095. cblack added a comment. Rename to kcm_users REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28154?vs=83079=83095 BRANCH arcpatch-D28154_1 REVISION DETAIL https://phabricator.kde.org/D28154 AFFECTED

D29823: Increasing the zoom effect and adding "launcherBounceLevel" config option

mvourlakos added a comment. Sorry I don't like the idea adding such option. REPOSITORY R878 Latte Dock REVISION DETAIL https://phabricator.kde.org/D29823 To: parsa, mvourlakos Cc: plasma-devel, acemond, Orage, LeGast00n, The-Feren-OS-Dev, cblack, kkoma, jraleigh, zachus, mbehrendt,

D29798: [wallpapers/image] Add button to open lock screen and SDDM KCMs

ngraham abandoned this revision. ngraham added a comment. All right, I'll abandon this revision and turn it into a Phab task: T13180: Make it easier to apply the wallpaper to the desktop(s), lock screen, and login screen all at once REPOSITORY R120

D7724: [ListItemBase] Open options menu on right click

broulik abandoned this revision. broulik added a comment. Will revisit on GitLab. Code has changed too much REPOSITORY R115 Plasma Audio Volume Applet REVISION DETAIL https://phabricator.kde.org/D7724 To: broulik, #plasma, #vdg, drosca Cc: ngraham, plasma-devel, Orage, LeGast00n,

D7724: [ListItemBase] Open options menu on right click

broulik added a comment. https://invent.kde.org/plasma/plasma-pa/-/merge_requests/1 REPOSITORY R115 Plasma Audio Volume Applet REVISION DETAIL https://phabricator.kde.org/D7724 To: broulik, #plasma, #vdg, drosca Cc: ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack,

Sunsetting PowerDevil's XRandrBrightness

Hi all, PowerDevil has two (three, if you count DDC) means of controlling screen brightness: * Writing to sysfs (/sys/class/backlight and /sys/class/leds) * Using XRandr output property XRandR is doing X calls and we have some lovely animated screen brightness change. However, the UPower

D29464: [applets/appmenu] Hide old menu before showing new menu

cblack closed this revision. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D29464 To: cblack, #plasma, #kwin, broulik Cc: broulik, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf,

D29395: Refactor MediaController

cblack added inline comments. INLINE COMMENTS > broulik wrote in Media.qml:6 > Make this a `QtObject` Seems to cause a segfault with the data source. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D29395 To: cblack, #plasma Cc: iasensio, broulik, trmdi,

D29395: Refactor MediaController

cblack updated this revision to Diff 83084. cblack added a comment. Guess that segfault wants to resolve itself :D REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29395?vs=83073=83084 BRANCH arcpatch-D29395_1 REVISION DETAIL

D29464: [applets/appmenu] Hide old menu before showing new menu

broulik requested changes to this revision. broulik added a comment. This revision now requires changes to proceed. Oops, I didnt't properly apply the patch. So yeah, the flickering for which I added this code is still there. REPOSITORY R120 Plasma Workspace REVISION DETAIL

D29464: [applets/appmenu] Hide old menu before showing new menu

cblack updated this revision to Diff 83083. cblack added a comment. Only do that on Wayland REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29464?vs=82041=83083 BRANCH arcpatch-D29464 REVISION DETAIL https://phabricator.kde.org/D29464

D28154: Add users KCM

cblack added inline comments. INLINE COMMENTS > broulik wrote in UserDetailsPage.qml:97 > Pfp? profile picture > broulik wrote in UserDetailsPage.qml:219 > Is this `RowLayout` needed? it's a convenience for the default alignment of a layout (`Qt.AlignVCenter | Qt.AlignLeft`) REPOSITORY

D29808: KSysGuard Network Plugin: Don't use std::regex to parse the network files.

sandsmark added inline comments. INLINE COMMENTS > sandsmark wrote in ConnectionMapping.cpp:167 > instead of 87 + 1, maybe have a `constexpr int lineLength = strlen("0: > 0100:0277 : > 0A : 00:

D29798: [wallpapers/image] Add button to open lock screen and SDDM KCMs

niccolove added a comment. I agree with a single KCM called wallpapers used to set all wallpaper in System Settings. Not a fan of having so many apply buttons, but I'm sure we can come up with something good. REPOSITORY R120 Plasma Workspace REVISION DETAIL

D29808: KSysGuard Network Plugin: Don't use std::regex to parse the network files.

sandsmark added inline comments. INLINE COMMENTS > ConnectionMapping.cpp:157 > +// Should be within the first 16 characters. > +size_t data_start = data.find(':'); > +if (data_start >= 16) { if we're going for ultra optimized, const everything I guess. >

D28154: Add users KCM

meven added a comment. So next step would be to shadow user_manager ? A naive ripgrep gave : $ rg user_manager .. ../kxmlgui/src/kbugreport.cpp 352:m_process->start(QStringLiteral("kcmshell5"), QStringList() << QStringLiteral("user_manager"));

D29808: KSysGuard Network Plugin: Don't use std::regex to parse the network files.

sandsmark added a comment. > It's all C code whereas the rest of the helper is C++. It also relies very heavily on magic numbers now. > I think a much simpler implementation would be to split each line on " ", select the fields we want and clean them up. I assume this is for