[Powerdevil] [Bug 339427] Screen goes off when watching video using VLC or YouTube.

https://bugs.kde.org/show_bug.cgi?id=339427 Valerio changed: What|Removed |Added CC||valeriod...@yahoo.it --- Comment #1 from Valerio

D5012: Delay notifications until desktop session has loaded

vpilo created this revision. vpilo added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Added program to wait for D-Bus Notifications services to register. Needed for Plasma to delay showing notifications until the desktop has fully loaded.

D5012: Delay notifications until desktop session has loaded

vpilo added a dependency: D4799: Delay notifications until desktop session has loaded. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D5012 To: vpilo Cc: graesslin, dfaure, davidedmundson, broulik, mck182, plasma-devel, progwolff, lesliezhai, ali-mohamed, jens

D4799: Delay notifications until desktop session has loaded

vpilo updated this revision to Diff 12377. vpilo marked an inline comment as done. vpilo added reviewers: Plasma, Plasma: Workspaces. vpilo added a comment. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. New patchset. I moved the waiti

D5012: Delay notifications until desktop session has loaded

vpilo added reviewers: Plasma, Plasma: Workspaces. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D5012 To: vpilo, #plasma, #plasma_workspaces Cc: graesslin, dfaure, davidedmundson, broulik, mck182, plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterbe

D5012: Delay notifications until desktop session has loaded

vpilo added reviewers: graesslin, dfaure, davidedmundson, broulik, mck182. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D5012 To: vpilo, #plasma, #plasma_workspaces, graesslin, dfaure, davidedmundson, broulik, mck182 Cc: graesslin, dfaure, davidedmundson, bro

D4799: Delay notifications until desktop session has loaded

vpilo added reviewers: davidedmundson, dfaure, broulik, graesslin, mck182. REPOSITORY R289 KNotifications REVISION DETAIL https://phabricator.kde.org/D4799 To: vpilo, #plasma, #plasma_workspaces, davidedmundson, dfaure, broulik, graesslin, mck182 Cc: plasma-devel, davidedmundson, dfaure, br

D5012: Delay notifications until desktop session has loaded

This revision was automatically updated to reflect the committed changes. Closed by commit R120:5bf889f7b94b: Add program to wait for D-Bus Notifications services to register. (authored by vpilo). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D5012?vs=12378&id=12609#toc REPOSITORY R120

D4799: Delay notifications until desktop session has loaded

vpilo added a comment. I pushed https://phabricator.kde.org/D5012; this rev is now testable. REPOSITORY R289 KNotifications REVISION DETAIL https://phabricator.kde.org/D4799 To: vpilo, #plasma, #plasma_workspaces, davidedmundson, dfaure, broulik, graesslin, mck182 Cc: plasma-devel, davi

D4799: Delay notifications until desktop session has loaded

vpilo added a comment. Ping [my excuses if it's not good practice to do so] REPOSITORY R289 KNotifications REVISION DETAIL https://phabricator.kde.org/D4799 To: vpilo, #plasma, #plasma_workspaces, davidedmundson, dfaure, broulik, graesslin, mck182 Cc: plasma-devel, davidedmundson,

D4799: Delay notifications until desktop session has loaded

This revision was automatically updated to reflect the committed changes. Closed by commit R289:2d40672b0c85: Do not remove queued notifications when the fd.o service starts. Also start the… (authored by vpilo). REPOSITORY R289 KNotifications CHANGES SINCE LAST UPDATE https://phabricator.kde

D18504: [plasma-nm/applet] Add right-click context menu to directly customize a connection

vpilo created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. vpilo requested review of this revision. REVISION SUMMARY Screenshot: https://imgur.com/MvnFdbW REPOSITORY R116 Plasma Network Management Applet BRANCH vpilo/contextMenu REVISION DETAIL

D18504: [plasma-nm/applet] Add right-click context menu to directly customize a connection

vpilo added a comment. In D18504#399351 , @GB_2 wrote: > Next time please upload images directly to Phabricator. 👍 REPOSITORY R116 Plasma Network Management Applet REVISION DETAIL https://phabricator.kde.org/D18504 To: vpilo, #vdg C

D18504: [plasma-nm/applet] Add right-click context menu to directly customize a connection

vpilo added a comment. In D18504#399356 , @abetts wrote: > Do you prefer a right click over a settings or 3-dot button? There's already very little space - I personally think it would become cluttered. An alternative might be a link whe

D18504: [plasma-nm/applet] Add right-click context menu to directly customize a connection

vpilo added a comment. In D18504#399653 , @jgrulich wrote: > Does it work for you on wayland? When the menu opens, it opens in the left top corner of the applet and cannot be closed unless you select any option, but that's probably wayland + qt

D18504: [plasma-nm/applet] Add right-click context menu to directly customize a connection

vpilo updated this revision to Diff 50248. vpilo marked 4 inline comments as done. vpilo added a comment. - Review comments REPOSITORY R116 Plasma Network Management Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18504?vs=50204&id=50248 BRANCH vpilo/contextMenu REVISION

D18504: [plasma-nm/applet] Add right-click context menu to directly customize a connection

vpilo added a comment. In D18504#399818 , @ngraham wrote: > In D18504#399653 , @jgrulich wrote: > > > Does it work for you on wayland? When the menu opens, it opens in the left top corner of the app

D18504: [plasma-nm/applet] Add right-click context menu to directly customize a connection

This revision was automatically updated to reflect the committed changes. Closed by commit R116:32d76665a694: [plasma-nm/applet] Add right-click context menu to directly customize a… (authored by vpilo). REPOSITORY R116 Plasma Network Management Applet CHANGES SINCE LAST UPDATE https://phabr

D18607: Add a popup search bar to the plasma-nm applet

vpilo created this revision. vpilo added reviewers: VDG, Plasma. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. vpilo requested review of this revision. REVISION SUMMARY If there are not enough connections to fill the scrollview, the search bar is not displayed by defa

D18607: Add a popup search bar to the plasma-nm applet

vpilo added a comment. Better just place it at the top above the the connections list, then? REPOSITORY R116 Plasma Network Management Applet REVISION DETAIL https://phabricator.kde.org/D18607 To: vpilo, #vdg, #plasma Cc: davidedmundson, plasma-devel, jraleigh, GB_2, ragreen, Pitel, Zren

D18607: Add a popup search bar to the plasma-nm applet

vpilo added a comment. In D18607#402261 , @jgrulich wrote: > Wouldn't be a small search bar placed next to the aiplane mode enough? That way it will not need additional vertical space. Hiding it when there is only a small amount of connections i

D18607: Add a popup search bar to the plasma-nm applet

vpilo added a comment. Here's my latest change, not submitted yet. | F6576947: short.png | F6576946: search.png | F6576945: long.png | | The differences are that I

D18621: Add again a button to manually scan for wireless networks

vpilo created this revision. vpilo added reviewers: VDG, Plasma, jgrulich, ngraham. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. vpilo requested review of this revision. REVISION SUMMARY The current 15 seconds timeout is really annoyingly long. NetworkManager ignore

D18607: Add a popup search bar to the plasma-nm applet

vpilo updated this revision to Diff 50567. vpilo added a comment. - review Up, not down No label Changed search label REPOSITORY R116 Plasma Network Management Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18607?vs=50518&id=50567 BRANCH vpilo/searchBar (branche

D18607: Add a popup search bar to the plasma-nm applet

vpilo added inline comments. INLINE COMMENTS > davidedmundson wrote in PopupDialog.qml:81 > The connection view changes height depending on the presence of a search bar > The search bar becomes visible depending on the height of the listview > > I would expect that to be giving binding loops whe

D18621: Add again a button to manually scan for wireless networks

vpilo added a comment. In D18621#402558 , @ngraham wrote: > In my opinion, this sort of thing is a "help the user feel in control" issue. We had the same sort of conversations in Discover back when it didn't have a manual "refresh updates" butto

D18607: Add a popup search bar to the plasma-nm applet

vpilo updated this revision to Diff 50582. vpilo added a comment. - New implementation with a toolbutton - Also added search reset on Esc keypress, on applet close, on search close I made a few tests and with all 3 toolbuttons (wifi/mobile/airplane) in place, also adding the search ba

D18607: Add a popup search bar to the plasma-nm applet

vpilo marked an inline comment as done. vpilo added inline comments. INLINE COMMENTS > PopupDialog.qml:45 > > -Toolbar { > -id: toolbar > +ColumnLayout { > +anchors.fill: parent This is to solve layouting issues when toggling the search bar. REPOSITORY R116 Plasma Ne

D18621: Add again a button to manually scan for wireless networks

vpilo updated this revision to Diff 50583. vpilo added a comment. - Review comments: change button tooltip REPOSITORY R116 Plasma Network Management Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18621?vs=50566&id=50583 BRANCH vpilo/refreshButton (branched from master)

D18621: Add again a button to manually scan for wireless networks

vpilo added a comment. Herald added 1 blocking reviewer(s): jgrulich. But there are legit cases where this timeout is way too long for no reason. If you close the popup and reopen it before the NM timeout expires, then the first request will fail, no attempt will be made at redoing the sc

D18621: Add again a button to manually scan for wireless networks

vpilo added a comment. In D18621#402717 , @jgrulich wrote: > In D18621#402712 , @vpilo wrote: > > > But there are legit cases where this timeout is way too long for no reason. > > > > If you clos

D18607: Add a popup search bar to the plasma-nm applet

This revision was automatically updated to reflect the committed changes. Closed by commit R116:684ee0c13bfb: Add a popup search bar to the plasma-nm applet (authored by vpilo). REPOSITORY R116 Plasma Network Management Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18607?vs=5

D18621: Add again a button to manually scan for wireless networks

vpilo added a comment. Let me be clear here - I don't really care about having this change merged, or having to rewrite it into something wholly different. I just want everyone to have their list of wifi networks to be up to date //when they need it//, that is, //when the nm applet is open//

D18504: [plasma-nm/applet] Add right-click context menu to directly customize a connection

vpilo added a comment. In D18504#404332 , @Codezela wrote: > small tweak we can make the connection name in the context menu bold header msybe > because it look like disabled now > it looks strange to me > what do u think I tried

D18745: Fix connections' context menus position and parent

vpilo created this revision. vpilo added reviewers: broulik, jgrulich. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. Herald added 1 blocking reviewer(s): jgrulich. vpilo requested review of this revision. REVISION SUMMARY A connection's context menu now opens only from

D18745: Fix connections' context menus position and parent

vpilo added a comment. In D18745#405458 , @jgrulich wrote: > This breaks connection details, try to expand details for some available connection. There appears to be some issue with heigh of the items. Oh, looks like I didn't test the lat

D18504: [plasma-nm/applet] Add right-click context menu to directly customize a connection

vpilo added a comment. In D18504#405816 , @ngraham wrote: > In D18504#405444 , @vpilo wrote: > > > In D18504#404332 , @Codezela wrote: > > > > > small t

D18745: Fix connections' context menus position and parent

vpilo updated this revision to Diff 50971. vpilo added a comment. Reworked. There already were lots of binding loops in this QML, stuff for another change REPOSITORY R116 Plasma Network Management Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18745?vs=50910&id=50971

D18745: Fix connections' context menus position and parent

This revision was automatically updated to reflect the committed changes. Closed by commit R116:5000fc973c6d: Fix connections' context menus position and parent (authored by vpilo). REPOSITORY R116 Plasma Network Management Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18745?

D18814: Clarify the label of the Shell KRunner.

vpilo created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. vpilo requested review of this revision. REVISION SUMMARY This makes sure the end result of the consequence of clicking this runner's results (running a shell command). REPOSITORY R120 Plasm

D18814: Clarify the label of the Shell KRunner.

vpilo added a comment. In D18814#407083 , @davidedmundson wrote: > It already says "Command line" in the group title on the left a console icon and unlike apps has the prefix "Run " "Run %1 in a shell" then? REPOSITORY R120 Plasma Wor

D18814: Clarify the label of the Shell KRunner.

vpilo abandoned this revision. vpilo added a comment. In D18814#407090 , @davidedmundson wrote: > What problem are we solving? > > If someone said "in a shell" I would expect to see a terminal (a shell) appear. That's not the case. I

D18621: Add again a button to manually scan for wireless networks

vpilo abandoned this revision. vpilo added a comment. I am abandoning this revision, since a button would be useless 95% of the time, thanks to NetworkManager. I'm submitting an automated retry mechanism. Later, I'd like to try adding some status icon showing when a new scan has been sta

D18896: [plasma-nm] Refresh wifi networks as fast as possible

vpilo created this revision. vpilo added reviewers: jgrulich, ngraham, Plasma. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. Herald added 1 blocking reviewer(s): jgrulich. vpilo requested review of this revision. REVISION SUMMARY The current 15s timeout is really annoy

D18896: [plasma-nm] Refresh wifi networks as fast as possible

vpilo added a comment. In D18896#409742 , @jgrulich wrote: > I would keep the 15s interval, everyone does 15s. Also from what I have read, every scan drops your connection for a while, which might be a problem for bad wifi drivers, where the sca

D18896: [plasma-nm] Refresh wifi networks as fast as possible

vpilo updated this revision to Diff 51525. vpilo added a comment. - Review comments, rework - Put the timer back to 15s - Removed trigger on start for QML timers: the timer now only runs if the current scan was successful - Don't use any more Qt 5.12 APIs REPOSITORY R116 Plasma Net

D18896: [plasma-nm] Refresh wifi networks as fast as possible

vpilo added inline comments. INLINE COMMENTS > jgrulich wrote in main.qml:78 > Since this is just one line function, you can use handler.requestScan() > directly as before. I did it for consistency, so in both files there's only one action. It's also useful for debugging. REPOSITORY R116 Pl

D18896: [plasma-nm] Refresh wifi networks as fast as possible

vpilo updated this revision to Diff 51677. vpilo added a comment. - Review comments REPOSITORY R116 Plasma Network Management Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18896?vs=51525&id=51677 BRANCH vpilo/wifiRefresh (branched from master) REVISION DETAIL https:/

D18896: [plasma-nm] Refresh wifi networks as fast as possible

This revision was automatically updated to reflect the committed changes. Closed by commit R116:1b49255d81f0: [plasma-nm] Refresh wifi networks as fast as possible (authored by vpilo). REPOSITORY R116 Plasma Network Management Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D188

D19013: [plasma-pa] Increase minimum size of expanded plasmoid

vpilo created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. vpilo requested review of this revision. REPOSITORY R115 Plasma Audio Volume Applet BRANCH vpilo/minimumSize REVISION DETAIL https://phabricator.kde.org/D19013 AFFECTED FILES applet/con

D19014: [plasma-pa] Add command to switch all applications to the selected device

vpilo created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. vpilo requested review of this revision. REVISION SUMMARY If you have many applications running (e.g. browser tabs) it's very annoying to manually switch them all to another device. With this c

D19013: [plasma-pa] Increase minimum size of expanded plasmoid

vpilo added a comment. In D19013#413022 , @ngraham wrote: > How about even bigger? In standalone form, there's no reason to skimp on size IMO. Thought there was some constraint :) I'll go crazy and say 18. REPOSITORY R115 Plasma Audio

D19013: [plasma-pa] Increase minimum size of expanded plasmoid

vpilo updated this revision to Diff 51862. vpilo added a comment. - Review comments REPOSITORY R115 Plasma Audio Volume Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D19013?vs=51698&id=51862 BRANCH vpilo/minimumSize REVISION DETAIL https://phabricator.kde.org/D19013

D19014: [plasma-pa] Add command to switch all applications to the selected device

vpilo added a comment. I would love some input about: - The wording of the menu entry. - The possibility of having an icon for the menu entry. REPOSITORY R115 Plasma Audio Volume Applet REVISION DETAIL https://phabricator.kde.org/D19014 To: vpilo, #plasma, broulik, drosca Cc: ngra

D19014: [plasma-pa] Add command to switch all applications to the selected device

vpilo updated this revision to Diff 51895. vpilo added a comment. - Review comments REPOSITORY R115 Plasma Audio Volume Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D19014?vs=51699&id=51895 BRANCH vpilo/switchStreamsToDevice (branched from master) REVISION DETAIL htt

D19014: [plasma-pa] Add command to switch all applications to the selected device

vpilo added inline comments. INLINE COMMENTS > ngraham wrote in ListItemBase.qml:306 > How about "Play all audio using this device" Better already; I'm also concerned whether the label should become too long. Let's see what happens. REPOSITORY R115 Plasma Audio Volume Applet REVISION DETAIL

D19013: [plasma-pa] Increase minimum size of expanded plasmoid

This revision was automatically updated to reflect the committed changes. Closed by commit R115:6e53d78136df: [plasma-pa] Increase minimum size of expanded plasmoid (authored by vpilo). REPOSITORY R115 Plasma Audio Volume Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D19013?vs

D19014: [plasma-pa] Add command to switch all applications to the selected device

vpilo added a reviewer: VDG. vpilo added inline comments. INLINE COMMENTS > ListItemBase.qml:315 > +menuItem.text = i18n("Play all audio via this device"); > +menuItem.icon = "audio-on" // or "audio-ready" // or > "audio-speakers-symbolic" > +menuI

D19014: [plasma-pa] Add command to switch all applications to the selected device

vpilo updated this revision to Diff 52146. vpilo added a comment. - Review comments - Add icons REPOSITORY R115 Plasma Audio Volume Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D19014?vs=51895&id=52146 BRANCH vpilo/switchStreamsToDevice (branched from master) REVISIO

D19014: [plasma-pa] Add command to switch all applications to the selected device

vpilo added a comment. INLINE COMMENTS > ngraham wrote in ListItemBase.qml:315 > Of those, I'd say `audio-ready` seems least inappropriate, but It almost > seems like we need a new icon that uses the speaker symbol with an arrow > somewhere in it to connote movement and change. I hoped I co

D19014: [plasma-pa] Add command to switch all applications to the selected device

vpilo added a comment. @ngraham If a new icon should be made, who could do it and how does the request process go? REPOSITORY R115 Plasma Audio Volume Applet REVISION DETAIL https://phabricator.kde.org/D19014 To: vpilo, #plasma, broulik, drosca, #vdg Cc: ngraham, plasma-devel, jraleigh,

D19014: [plasma-pa] Add command to switch all applications to the selected device

vpilo added a comment. In D19014#417976 , @ngraham wrote: > In D19014#417906 , @vpilo wrote: > > > @ngraham If a new icon should be made, who could do it and how does the request process go? > >

D19014: [plasma-pa] Add command to switch all applications to the selected device

vpilo added a comment. Excellent. Sorry I had no time to follow up for the icon creation yet. REPOSITORY R115 Plasma Audio Volume Applet BRANCH vpilo/switchStreamsToDevice (branched from master) REVISION DETAIL https://phabricator.kde.org/D19014 To: vpilo, #plasma, broulik, drosca, #v

D19014: [plasma-pa] Add command to switch all applications to the selected device

This revision was automatically updated to reflect the committed changes. Closed by commit R115:61259acda87b: [plasma-pa] Add command to switch all applications to the selected device (authored by vpilo). REPOSITORY R115 Plasma Audio Volume Applet CHANGES SINCE LAST UPDATE https://phabricato

D5012: Delay notifications until desktop session has loaded

vpilo added a comment. In https://phabricator.kde.org/D5012#110815, @lbeltrame wrote: > Months later, it turns out that notify-osd ships a org.freedesktop.Notifications.service already, so it conflicts with the one made by this patch. I wouldn't know what course of action would b

[Powerdevil] [Bug 340892] System Settings's Power management complains about power management service

https://bugs.kde.org/show_bug.cgi?id=340892 Valerio De Angelis changed: What|Removed |Added CC||valeriod...@yahoo.it --- Comment #3 from

Re: Review Request 125248: Display name of process which blocks umount / eject

<https://git.reviewboard.kde.org//r/125248/#fcomment437> Could be possible to put the applications list in bold? - Valerio De Angelis On Sept. 16, 2015, 6:07 p.m., Igor Poboiko wrote: > > --- > This is an automatically generated e-mail. To