T8066: Fix Bug 390830 = Prevent indexing of Plasma vaults and other fuse filesystems

[Stefan Brüns]

bruns added a comment.


  If you want a manual workaround, just add the vault path to excludeFolders.
  If you want a proper fix, handle encfs etc. properly in baloo.

TASK DETAIL
  https://phabricator.kde.org/T8066

To: michaelh, bruns
Cc: smithjd, bruns, nicolasfella, ivan, plasma-devel, alexeymin, michaelh, 
ashaposhnikov, astippich, spoorun, abrahams

T8066: Fix Bug 390830 = Prevent indexing of Plasma vaults and other fuse filesystems

[James Smith]

smithjd added a comment.


  What's wrong with a .balooignore file in the root of the vault? Tracker uses 
.trackerignore.

TASK DETAIL
  https://phabricator.kde.org/T8066

To: michaelh, smithjd
Cc: smithjd, bruns, nicolasfella, ivan, plasma-devel, alexeymin, michaelh, 
ashaposhnikov, astippich, spoorun, abrahams

[Plasma Workspace Wallpapers] [Bug 347532] preferencias de escritorio no permite establecer fondo de pantalla ni individual ni en presentacion solo las del sistenma

[Andrew Crouthamel]

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

Andrew Crouthamel  changed:

   What|Removed |Added

   Keywords||triaged

--- Comment #5 from Andrew Crouthamel  ---
Dear Bug Submitter,

This bug has been in NEEDSINFO status with no change for at least 15 days.
Please provide the requested information as soon as possible and set the bug
status as REPORTED. Due to regular bug tracker maintenance, if the bug is still
in NEEDSINFO status with no change in 30 days, the bug will be closed as
RESOLVED > WORKSFORME due to lack of needed information.

For more information about our bug triaging procedures please read the wiki
located here: https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

If you have already provided the requested information, please set the bug
status as REPORTED so that the KDE team knows that the bug is ready to be
confirmed.

Thank you for helping us make KDE software even better for everyone!

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Breeze] [Bug 356902] Starters are visually indistinguishable from listed tasks in the task manager widget on breeze light

[Andrew Crouthamel]

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

Andrew Crouthamel  changed:

   What|Removed |Added

   Keywords||triaged

--- Comment #3 from Andrew Crouthamel  ---
Dear Bug Submitter,

This bug has been in NEEDSINFO status with no change for at least 15 days.
Please provide the requested information as soon as possible and set the bug
status as REPORTED. Due to regular bug tracker maintenance, if the bug is still
in NEEDSINFO status with no change in 30 days, the bug will be closed as
RESOLVED > WORKSFORME due to lack of needed information.

For more information about our bug triaging procedures please read the wiki
located here: https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

If you have already provided the requested information, please set the bug
status as REPORTED so that the KDE team knows that the bug is ready to be
confirmed.

Thank you for helping us make KDE software even better for everyone!

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Breeze] [Bug 346369] the password feild is not set to empty after wrong password attempt

[Andrew Crouthamel]

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

Andrew Crouthamel  changed:

   What|Removed |Added

   Keywords||triaged

--- Comment #4 from Andrew Crouthamel  ---
Dear Bug Submitter,

This bug has been in NEEDSINFO status with no change for at least 15 days.
Please provide the requested information as soon as possible and set the bug
status as REPORTED. Due to regular bug tracker maintenance, if the bug is still
in NEEDSINFO status with no change in 30 days, the bug will be closed as
RESOLVED > WORKSFORME due to lack of needed information.

For more information about our bug triaging procedures please read the wiki
located here: https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

If you have already provided the requested information, please set the bug
status as REPORTED so that the KDE team knows that the bug is ready to be
confirmed.

Thank you for helping us make KDE software even better for everyone!

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Powerdevil] [Bug 334411] Screen Energy Saving timeout on dual-monitor system starting one after another with timeout delay

[Andrew Crouthamel]

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

Andrew Crouthamel  changed:

   What|Removed |Added

   Keywords||triaged

--- Comment #3 from Andrew Crouthamel  ---
Dear Bug Submitter,

This bug has been in NEEDSINFO status with no change for at least 15 days.
Please provide the requested information as soon as possible and set the bug
status as REPORTED. Due to regular bug tracker maintenance, if the bug is still
in NEEDSINFO status with no change in 30 days, the bug will be closed as
RESOLVED > WORKSFORME due to lack of needed information.

For more information about our bug triaging procedures please read the wiki
located here: https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

If you have already provided the requested information, please set the bug
status as REPORTED so that the KDE team knows that the bug is ready to be
confirmed.

Thank you for helping us make KDE software even better for everyone!

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Powerdevil] [Bug 341619] Screen Brightness does not change

[Andrew Crouthamel]

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

Andrew Crouthamel  changed:

   What|Removed |Added

   Keywords||triaged

--- Comment #3 from Andrew Crouthamel  ---
Dear Bug Submitter,

This bug has been in NEEDSINFO status with no change for at least 15 days.
Please provide the requested information as soon as possible and set the bug
status as REPORTED. Due to regular bug tracker maintenance, if the bug is still
in NEEDSINFO status with no change in 30 days, the bug will be closed as
RESOLVED > WORKSFORME due to lack of needed information.

For more information about our bug triaging procedures please read the wiki
located here: https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

If you have already provided the requested information, please set the bug
status as REPORTED so that the KDE team knows that the bug is ready to be
confirmed.

Thank you for helping us make KDE software even better for everyone!

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Breeze] [Bug 350682] Double-click on GTK widgets initiates window movement, breaks control of the widget

[Andrew Crouthamel]

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

Andrew Crouthamel  changed:

   What|Removed |Added

   Keywords||triaged

--- Comment #11 from Andrew Crouthamel  ---
Dear Bug Submitter,

This bug has been in NEEDSINFO status with no change for at least 15 days.
Please provide the requested information as soon as possible and set the bug
status as REPORTED. Due to regular bug tracker maintenance, if the bug is still
in NEEDSINFO status with no change in 30 days, the bug will be closed as
RESOLVED > WORKSFORME due to lack of needed information.

For more information about our bug triaging procedures please read the wiki
located here: https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

If you have already provided the requested information, please set the bug
status as REPORTED so that the KDE team knows that the bug is ready to be
confirmed.

Thank you for helping us make KDE software even better for everyone!

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Powerdevil] [Bug 349226] battery updated not updated after suspend to memory

[Andrew Crouthamel]

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

Andrew Crouthamel  changed:

   What|Removed |Added

   Keywords||triaged

--- Comment #5 from Andrew Crouthamel  ---
Dear Bug Submitter,

This bug has been in NEEDSINFO status with no change for at least 15 days.
Please provide the requested information as soon as possible and set the bug
status as REPORTED. Due to regular bug tracker maintenance, if the bug is still
in NEEDSINFO status with no change in 30 days, the bug will be closed as
RESOLVED > WORKSFORME due to lack of needed information.

For more information about our bug triaging procedures please read the wiki
located here: https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

If you have already provided the requested information, please set the bug
status as REPORTED so that the KDE team knows that the bug is ready to be
confirmed.

Thank you for helping us make KDE software even better for everyone!

-- 
You are receiving this mail because:
You are the assignee for the bug.

D12498: Fully remove `Application Name` from Details panel

[Nathaniel Graham]

ngraham added a comment.


  It landed; you wanna finish this up now?

REPOSITORY
  R121 Policykit (Polkit) KDE Agent

REVISION DETAIL
  https://phabricator.kde.org/D12498

To: sharvey, bruns, ngraham, davidedmundson
Cc: davidedmundson, bruns, ngraham, plasma-devel, ragreen, Pitel, ZrenBot, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D15749: Add tooltips to krunner configure and close buttons

[Andrew Crouthamel]

acrouthamel added a comment.


  In D15749#331860 , @ngraham wrote:
  
  > would you like to do the honors and land this for him, @acrouthamel?
  
  
  Yes boss... Always roping me into something new. :)

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D15749

To: slenz, #vdg, acrouthamel, ngraham
Cc: ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D15749: Add tooltips to krunner configure and close buttons

[Andrew Crouthamel]

This revision was automatically updated to reflect the committed changes.
Closed by commit R120:f57460a61643: Add tooltips to krunner configure and close 
buttons (authored by slenz, committed by acrouthamel).

REPOSITORY
  R120 Plasma Workspace

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D15749?vs=42319&id=42332

REVISION DETAIL
  https://phabricator.kde.org/D15749

AFFECTED FILES
  lookandfeel/contents/runcommand/RunCommand.qml

To: slenz, #vdg, acrouthamel, ngraham
Cc: ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D15304: [KCM] Port label-bearing controls to QQC2 to fix fractional scaling support

[Nathaniel Graham]

ngraham updated this revision to Diff 42329.
ngraham added a comment.


  Port (almost) entirely to QQC2

REPOSITORY
  R115 Plasma Audio Volume Applet

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D15304?vs=41077&id=42329

BRANCH
  arcpatch-D15304

REVISION DETAIL
  https://phabricator.kde.org/D15304

AFFECTED FILES
  src/kcm/package/contents/ui/Advanced.qml
  src/kcm/package/contents/ui/Applications.qml
  src/kcm/package/contents/ui/CardListItem.qml
  src/kcm/package/contents/ui/DeviceComboBox.qml
  src/kcm/package/contents/ui/DeviceListItem.qml
  src/kcm/package/contents/ui/Devices.qml
  src/kcm/package/contents/ui/Header.qml
  src/kcm/package/contents/ui/StreamListItem.qml
  src/kcm/package/contents/ui/VolumeSlider.qml
  src/kcm/package/contents/ui/main.qml

To: ngraham, #plasma, drosca
Cc: davidedmundson, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D15749: Add tooltips to krunner configure and close buttons

[Nathaniel Graham]

ngraham added a comment.


  Since Silas lacks commit rights (I found this out by searching on 
https://websvn.kde.org/trunk/kde-common/accounts?view=markup), would you like 
to do the honors and land this for him, @acrouthamel? Make sure the git 
authorship is set properly first. Land on master please since there are new 
strings.
  
  Documentation is here: 
https://community.kde.org/Infrastructure/Phabricator#Landing_someone_else.27s_diff

REPOSITORY
  R120 Plasma Workspace

BRANCH
  krunner-tootlips (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D15749

To: slenz, #vdg, acrouthamel, ngraham
Cc: ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D15749: Add tooltips to krunner configure and close buttons

[Andrew Crouthamel]

acrouthamel accepted this revision.
acrouthamel added a comment.


  Ooh my first time being added as reviewer. ;)

REPOSITORY
  R120 Plasma Workspace

BRANCH
  krunner-tootlips (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D15749

To: slenz, #vdg, acrouthamel, ngraham
Cc: ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D15749: Add tooltips to krunner configure and close buttons

[Silas Lenz]

slenz updated this revision to Diff 42319.
slenz added a comment.


  Remove "KRunner" from configure tooltip

REPOSITORY
  R120 Plasma Workspace

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D15749?vs=42318&id=42319

BRANCH
  krunner-tootlips (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D15749

AFFECTED FILES
  lookandfeel/contents/runcommand/RunCommand.qml

To: slenz, #vdg, acrouthamel
Cc: ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D15749: Add tooltips to krunner configure and close buttons

[Nathaniel Graham]

ngraham added inline comments.

INLINE COMMENTS

> ngraham wrote in RunCommand.qml:64
> Hmm, good question. Let me ask #VDG .

For now, let's just go with "Configure..." and we'll sort out the user-facing 
string at another time so we don't derail your patch. :)

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D15749

To: slenz, #vdg, acrouthamel
Cc: ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D14830: Add "Configure Displays" context menu entry on the desktop

[gregormi]

gregormi added a comment.


  In D14830#313645 , @davidedmundson 
wrote:
  
  > I am still waiting for a single justification.
  
  
  In https://phabricator.kde.org/T6875, under "Who should be our primary target 
groups?" we have "Windows 7 Power Users". Those users will benefit from this 
change (because there, the well-known way to reach the configure screens dialog 
is to right click on the desktop).

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D14830

To: broulik, #plasma, #vdg, fischbach, harmathy, davidedmundson
Cc: gregormi, mart, davidedmundson, abetts, romangg, fabianr, ngraham, 
plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, 
sebas, apol

D15749: Add tooltips to krunner configure and close buttons

[Nathaniel Graham]

ngraham added inline comments.

INLINE COMMENTS

> slenz wrote in RunCommand.qml:64
> You could argue KRunner is not used as a user-facing name anywhere (Setting 
> calls it `Plasma Search` and `Search Bar` depending on where you look). Might 
> be less confusing to just keep it at `Configure...`?

Hmm, good question. Let me ask #VDG .

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D15749

To: slenz, #vdg, acrouthamel
Cc: ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D15749: Add tooltips to krunner configure and close buttons

[Silas Lenz]

slenz added inline comments.

INLINE COMMENTS

> ngraham wrote in RunCommand.qml:64
> Needs ellipses; see 
> https://hig.kde.org/style/writing/labels.html#using-ellipses-in-labels
> 
> How about "Configure KRunner..." ?

You could argue KRunner is not used as a user-facing name anywhere (Setting 
calls it `Plasma Search` and `Search Bar` depending on where you look). Might 
be less confusing to just keep it at `Configure...`?

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D15749

To: slenz, #vdg, acrouthamel
Cc: ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D15749: Add tooltips to krunner configure and close buttons

[Silas Lenz]

slenz updated this revision to Diff 42318.
slenz added a comment.


  Add ellipses and KRunner to configure tooltip.

REPOSITORY
  R120 Plasma Workspace

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D15749?vs=42316&id=42318

BRANCH
  krunner-tootlips (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D15749

AFFECTED FILES
  lookandfeel/contents/runcommand/RunCommand.qml

To: slenz, #vdg, acrouthamel
Cc: ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D15749: Add tooltips to krunner configure and close buttons

[Nathaniel Graham]

ngraham added a reviewer: acrouthamel.
ngraham added a comment.


  Wow, that was fast. One minor suggestion:

INLINE COMMENTS

> RunCommand.qml:64
>  visible: runnerWindow.canConfigure
> +tooltip: i18nd("plasma_lookandfeel_org.kde.lookandfeel", 
> "Configure")
>  }

Needs ellipses; see 
https://hig.kde.org/style/writing/labels.html#using-ellipses-in-labels

How about "Configure KRunner..." ?

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D15749

To: slenz, #vdg, acrouthamel
Cc: ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D15749: Add tooltips to krunner configure and close buttons

[Silas Lenz]

slenz created this revision.
slenz added a reviewer: VDG.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
slenz requested review of this revision.

REVISION SUMMARY
  Shows tooltips for buttons left and right of the search box.
  
  FEATURE: 398943

TEST PLAN
  Manually test appearance of tooltips, unlikely to cause new issues.

REPOSITORY
  R120 Plasma Workspace

BRANCH
  krunner-tootlips (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D15749

AFFECTED FILES
  lookandfeel/contents/runcommand/RunCommand.qml

To: slenz, #vdg
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D15514: [libbreezecommon] Use box blur instead of FFT blur

[Vlad Zagorodniy]

zzag planned changes to this revision.
zzag added a comment.


  Follow https://www.w3.org/TR/SVG11/filters.html#feGaussianBlurElement

REPOSITORY
  R31 Breeze

REVISION DETAIL
  https://phabricator.kde.org/D15514

To: zzag, #kwin, #plasma
Cc: cfeck, ngraham, abetts, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, 
ali-mohamed, jensreuterberg, sebas, apol, mart

D15644: Provide option to hide menu bar for Ksysguard

[Nathaniel Graham]

ngraham added a comment.


  Actually, reading over this again, is it really necessary to add a 
`showMessage` parameter to `toggleShowMenuBar`? In general bool-only arguments 
are frowned upon because they're not very readable; enums are preferred in 
their place. But do we even need that parameter in the first place? I don't see 
that it's ever even set to false anywhere.

REVISION DETAIL
  https://phabricator.kde.org/D15644

To: lsartorelli, ngraham, #plasma, #frameworks
Cc: acrouthamel, ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D15093: Add WireGuard capability.

[Nathaniel Graham]

ngraham added a comment.


  In D15093#331505 , @andersonbruce 
wrote:
  
  > - Add entry widget background color change based on entry validity
  > - Change QSpinBoxes back to QLineEdits
  
  
  Awesome, +1 from me on these changes!

REPOSITORY
  R116 Plasma Network Management Applet

REVISION DETAIL
  https://phabricator.kde.org/D15093

To: andersonbruce, #plasma, jgrulich, pino
Cc: acrouthamel, K900, pino, lbeltrame, ngraham, plasma-devel, ragreen, Pitel, 
ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D15644: Provide option to hide menu bar for Ksysguard

[Luca Sartorelli]

lsartorelli added a comment.


  Thank you very much,
  
  I am so happy and proud for my first patch.
  
  Here my details:
  Name: Luca 
  Surname: Sartorelli 
  Mail: dj3...@gmail.com
  
  And here is the patch for gwenview: https://phabricator.kde.org/D15747
  
  Just added you as reviewer 2 secs ago

REVISION DETAIL
  https://phabricator.kde.org/D15644

To: lsartorelli, ngraham, #plasma, #frameworks
Cc: acrouthamel, ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

[Powerdevil] [Bug 350365] Battery monitor in tray randomly shows there is no battery

[Nate Graham]

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

--- Comment #30 from Nate Graham  ---
All right, I'll butt out now. :)

-- 
You are receiving this mail because:
You are the assignee for the bug.

D15644: Provide option to hide menu bar for Ksysguard

[Nathaniel Graham]

ngraham accepted this revision.
ngraham added a comment.
This revision is now accepted and ready to land.


  Wonderful. I think the warning is good enough for now until we can come up 
with a better way of handling this. Code looks good! Can you provide your full 
name and email address so I can land this patch with proper authorship 
information?
  
  Now that you're an expert on creating warning messages when the menu bar is 
hidden, would you like to try your hand at doing the same thing for Gwenview? 
https://bugs.kde.org/show_bug.cgi?id=210620 ;-)
  
  Any concerns from #plasma  or 
#frameworks  folks before I land 
this?

REVISION DETAIL
  https://phabricator.kde.org/D15644

To: lsartorelli, ngraham, #plasma, #frameworks
Cc: acrouthamel, ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

[Powerdevil] [Bug 350365] Battery monitor in tray randomly shows there is no battery

[Kai Uwe Broulik]

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

--- Comment #29 from Kai Uwe Broulik  ---
I suspect it's an update bug somewhere, screen locker starts and then reads the
information fresh on start. We have found an issue in the proxy model we use
for sorting batteries but that seems unlikely to just hide them all, especially
not in the tooltip.

-- 
You are receiving this mail because:
You are the assignee for the bug.

D15744: [weather applet] Fix unwanted ruling of implicit icon sizes on displayed size

[Friedrich W. H. Kossebau]

This revision was automatically updated to reflect the committed changes.
Closed by commit R114:a49908b6eaef: [weather applet] Fix unwanted ruling of 
implicit icon sizes on displayed size (authored by kossebau).

REPOSITORY
  R114 Plasma Addons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D15744?vs=42309&id=42310

REVISION DETAIL
  https://phabricator.kde.org/D15744

AFFECTED FILES
  applets/weather/package/contents/ui/ForecastView.qml
  applets/weather/package/contents/ui/TopPanel.qml

To: kossebau, broulik
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

[Powerdevil] [Bug 350365] Battery monitor in tray randomly shows there is no battery

[Rolf Eike Beer]

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

Rolf Eike Beer  changed:

   What|Removed |Added

Version|5.3.2   |5.12.5

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Powerdevil] [Bug 350365] Battery monitor in tray randomly shows there is no battery

[Rolf Eike Beer]

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

--- Comment #28 from Rolf Eike Beer  ---
So why does it work on the lockscreen, does that read it's information from
somewhere else?

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Powerdevil] [Bug 350365] Battery monitor in tray randomly shows there is no battery

[Nate Graham]

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

Nate Graham  changed:

   What|Removed |Added

 CC||n...@kde.org

--- Comment #27 from Nate Graham  ---
Who owns upower? Should we close this as RESOLVED UPSTREAM with a request to
file a bug to upower?

-- 
You are receiving this mail because:
You are the assignee for the bug.

D15744: [weather applet] Fix unwanted ruling of implicit icon sizes on displayed size

[Friedrich W. H. Kossebau]

kossebau updated this revision to Diff 42309.
kossebau marked an inline comment as done.
kossebau added a comment.


  - use Repeater.count & Repeater.itemAt(i)
  - drop additional property preferredWidth on cell items, calculate from 
normal properties

REPOSITORY
  R114 Plasma Addons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D15744?vs=42299&id=42309

BRANCH
  fiximpliciticonsizeruling

REVISION DETAIL
  https://phabricator.kde.org/D15744

AFFECTED FILES
  applets/weather/package/contents/ui/ForecastView.qml
  applets/weather/package/contents/ui/TopPanel.qml

To: kossebau, broulik
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D15738: [Fonts KCM] remove filler words from anti-aliasing settings' labels

[Andres Betts]

abetts added a comment.


  +1

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D15738

To: ngraham, #vdg, #plasma
Cc: abetts, svenmauch, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, 
ali-mohamed, jensreuterberg, sebas, apol, mart

D15744: [weather applet] Fix unwanted ruling of implicit icon sizes on displayed size

[Friedrich W. H. Kossebau]

kossebau added a comment.


  Thanks for look, Will update based on feedback later today.

INLINE COMMENTS

> broulik wrote in ForecastView.qml:38
> `Layout.preferredWidth` defaults to `implicitWidth`, see [1] on the 
> "preference order"
> 
> [1] 
> https://doc.qt.io/qt-5/qtquicklayouts-overview.html#specifying-preferred-size

I  did this code to spare some `if Layout && Layout.preferredWidth && 
Layout.preferredWidth > 0 { use Layout.preferredWidth } else { else use 
implicitWidth) in the calculation, seems that was not obvious outside my brain 
:)
Guess I should then not do that optimization, no big loss, will change.

> broulik wrote in ForecastView.qml:119
> You can just do `children[i].Layout.preferredWidth`, no need for a "proxy" 
> property
> 
> Also, shouldn't it rather be
> 
>   for (var i = 0; i < repeater.count; ++i) {
>   var child = repeater.itemAt(i);
>   ...
>   }
> 
> Also I *think* that should auto-propagate, no?

Had not yet tried to look at repeater childs, let me try that.

At least from what I remember about testing any auto-propagation from repeater 
& loader, this did not work as expected, so I turned to do all explicitly. 
Though not exactly sure what you mean with "that"?

REPOSITORY
  R114 Plasma Addons

REVISION DETAIL
  https://phabricator.kde.org/D15744

To: kossebau, broulik
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D15744: [weather applet] Fix unwanted ruling of implicit icon sizes on displayed size

[Kai Uwe Broulik]

broulik requested changes to this revision.
broulik added a comment.
This revision now requires changes to proceed.


  I think all of this could be cleaned up significantly

INLINE COMMENTS

> ForecastView.qml:38
>  PlasmaComponents.Label {
> +readonly property int preferredWidth: implicitWidth
> +

`Layout.preferredWidth` defaults to `implicitWidth`, see [1] on the "preference 
order"

[1] 
https://doc.qt.io/qt-5/qtquicklayouts-overview.html#specifying-preferred-size

> ForecastView.qml:119
> +}
> +mw = Math.max(mw, children[i].preferredWidth);
>  }

You can just do `children[i].Layout.preferredWidth`, no need for a "proxy" 
property

Also, shouldn't it rather be

  for (var i = 0; i < repeater.count; ++i) {
  var child = repeater.itemAt(i);
  ...
  }

Also I *think* that should auto-propagate, no?

REPOSITORY
  R114 Plasma Addons

REVISION DETAIL
  https://phabricator.kde.org/D15744

To: kossebau, broulik
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D15744: [weather applet] Fix unwanted ruling of implicit icon sizes on displayed size

[Friedrich W. H. Kossebau]

kossebau created this revision.
kossebau added a reviewer: broulik.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
kossebau requested review of this revision.

REVISION SUMMARY
  BUG: 398960
  FIXED-IN: 5.14.0

TEST PLAN
  Switching icon sets between newaita, Breeze, or Oxygen no longer results in
  different (too large) icon sizes to be used.

REPOSITORY
  R114 Plasma Addons

BRANCH
  fiximpliciticonsizeruling

REVISION DETAIL
  https://phabricator.kde.org/D15744

AFFECTED FILES
  applets/weather/package/contents/ui/ForecastView.qml
  applets/weather/package/contents/ui/TopPanel.qml

To: kossebau, broulik
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

Plasma 5.12.7 LTS

[Jonathan Riddell]

Plasma 5.12.7 LTS is now released
https://www.kde.org/announcements/plasma-5.12.7.php

[Powerdevil] [Bug 350365] Battery monitor in tray randomly shows there is no battery

[Kai Uwe Broulik]

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

Kai Uwe Broulik  changed:

   What|Removed |Added

 CC||k...@privat.broulik.de

--- Comment #26 from Kai Uwe Broulik  ---
Given the battery isn't even shown in upower I suspect a bug there, we show
exactly what Upower reports to us.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Powerdevil] [Bug 350365] Battery monitor in tray randomly shows there is no battery

[Marco Richetta]

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

--- Comment #25 from Marco Richetta  ---
Created attachment 115218
  --> https://bugs.kde.org/attachment.cgi?id=115218&action=edit
Battery detected

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Powerdevil] [Bug 350365] Battery monitor in tray randomly shows there is no battery

[Marco Richetta]

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

--- Comment #24 from Marco Richetta  ---
Created attachment 115217
  --> https://bugs.kde.org/attachment.cgi?id=115217&action=edit
Battery not detected

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Powerdevil] [Bug 350365] Battery monitor in tray randomly shows there is no battery

[Marco Richetta]

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

Marco Richetta  changed:

   What|Removed |Added

 CC||marcoriche...@gmail.com

--- Comment #23 from Marco Richetta  ---
Same problem here with 
- Kubuntu 18.04 
- KDE Frameworks 5.44
- KDE Plasma 5.12.6
- Kernel 4.15.0-34-generic
- UPower is at version 0.99.7.

Reproducible: Almost always

Steps to reproduce:
1. After logon, unplug and plug the battery.
2. Close session and login again.
3. Battery is showed by plasmoid and detected by UPower.

*Sometimes battery is detected before the session is closed.

upower-d output:

BEFORE BEING DETECTED
=

Device: /org/freedesktop/UPower/devices/line_power_ADP1
  native-path:  ADP1
  power supply: yes
  updated:  mar 25 sep 2018 06:56:19 -03 (191 seconds ago)
  has history:  no
  has statistics:   no
  line-power
warning-level:   none
online:  no
icon-name:  'ac-adapter-symbolic'

Device: /org/freedesktop/UPower/devices/DisplayDevice
  power supply: no
  updated:  mar 25 sep 2018 06:56:19 -03 (191 seconds ago)
  has history:  no
  has statistics:   no
  unknown
warning-level:   none
icon-name:  'battery-missing-symbolic'

Daemon:
  daemon-version:  0.99.7
  on-battery:  no
  lid-is-closed:   no
  lid-is-present:  yes
  critical-action: HybridSleep

AFTER BEING DETECTED


Device: /org/freedesktop/UPower/devices/line_power_ADP1
  native-path:  ADP1
  power supply: yes
  updated:  mar 25 sep 2018 06:56:19 -03 (988 seconds ago)
  has history:  no
  has statistics:   no
  line-power
warning-level:   none
online:  no
icon-name:  'ac-adapter-symbolic'

Device: /org/freedesktop/UPower/devices/battery_BAT1
  native-path:  BAT1
  vendor:   -Real Battery 1-
  model:CRB Battery 1
  serial:   Battery 1
  power supply: yes
  updated:  mar 25 sep 2018 07:11:47 -03 (60 seconds ago)
  has history:  yes
  has statistics:   yes
  battery
present: yes
rechargeable:yes
state:   charging
warning-level:   none
energy:  3,118 Wh
energy-empty:0 Wh
energy-full: 3,325 Wh
energy-full-design:  4,4 Wh
energy-rate: 0 W
percentage:  93%
capacity:75,5682%
icon-name:  'battery-full-charging-symbolic'
  History (charge):
1537870306  93,000  charging
1537870306  0,000   unknown
  History (rate):
1537870306  0,000   unknown

Device: /org/freedesktop/UPower/devices/DisplayDevice
  power supply: yes
  updated:  mar 25 sep 2018 07:11:46 -03 (61 seconds ago)
  has history:  no
  has statistics:   no
  battery
present: yes
state:   charging
warning-level:   none
energy:  3,118 Wh
energy-full: 3,325 Wh
energy-rate: 0 W
percentage:  93%
icon-name:  'battery-full-charging-symbolic'

If I can reproduce this behaviour it would be useful
to find the logs of this activity, but I don't know where to look.

*On x.sessionerrors it's shown only one "Unable to assign [undefined] to int"
error.

-- 
You are receiving this mail because:
You are the assignee for the bug.

D15644: Provide option to hide menu bar for Ksysguard

[Luca Sartorelli]

lsartorelli updated this revision to Diff 42298.
lsartorelli added a comment.


  Added remainder message box with keyboard shortcut, to have back the menu bar.

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D15644?vs=42101&id=42298

REVISION DETAIL
  https://phabricator.kde.org/D15644

AFFECTED FILES
  gui/ksysguard.cpp
  gui/ksysguard.h

To: lsartorelli, ngraham, #plasma, #frameworks
Cc: acrouthamel, ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D15520: Upgrade SimpleIpV4AddressValidator and SimpleIpV6AddressValidator

[Bruce Anderson]

andersonbruce updated this revision to Diff 42297.
andersonbruce added a comment.


  - Fix review comments

REPOSITORY
  R116 Plasma Network Management Applet

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D15520?vs=42281&id=42297

BRANCH
  IP4/6validatorsUpdate

REVISION DETAIL
  https://phabricator.kde.org/D15520

AFFECTED FILES
  CMakeLists.txt
  libs/editor/CMakeLists.txt
  libs/editor/simpleiplistvalidator.cpp
  libs/editor/simpleiplistvalidator.h
  libs/editor/simpleipv4addressvalidator.cpp
  libs/editor/simpleipv4addressvalidator.h
  libs/editor/simpleipv6addressvalidator.cpp
  libs/editor/simpleipv6addressvalidator.h
  tests/CMakeLists.txt
  tests/simpleiplisttest.cpp
  tests/simpleipv4test.cpp
  tests/simpleipv6test.cpp

To: andersonbruce, jgrulich, pino
Cc: ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D15514: [libbreezecommon] Use box blur instead of FFT blur

[Vlad Zagorodniy]

zzag updated this revision to Diff 42295.
zzag added a comment.


  Comment on why shadows are a little bit lighter.

REPOSITORY
  R31 Breeze

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D15514?vs=41667&id=42295

BRANCH
  zzag/box-shadow-helper-box-blur

REVISION DETAIL
  https://phabricator.kde.org/D15514

AFFECTED FILES
  CMakeLists.txt
  cmake/Modules/FindFFTW.cmake
  libbreezecommon/CMakeLists.txt
  libbreezecommon/breezeboxshadowhelper.cpp

To: zzag, #kwin, #plasma
Cc: cfeck, ngraham, abetts, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, 
ali-mohamed, jensreuterberg, sebas, apol, mart

D15093: Add WireGuard capability.

[Bruce Anderson]

andersonbruce updated this revision to Diff 42290.
andersonbruce added a comment.


  - Add entry widget background color change based on entry validity
  - Change QSpinBoxes back to QLineEdits

REPOSITORY
  R116 Plasma Network Management Applet

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D15093?vs=42118&id=42290

BRANCH
  Feature/AddWireGuardVPN

REVISION DETAIL
  https://phabricator.kde.org/D15093

AFFECTED FILES
  libs/models/kcmidentitymodel.cpp
  vpn/CMakeLists.txt
  vpn/wireguard/CMakeLists.txt
  vpn/wireguard/nm-wireguard-service.h
  vpn/wireguard/plasmanetworkmanagement_wireguardui.desktop
  vpn/wireguard/wireguard.cpp
  vpn/wireguard/wireguard.h
  vpn/wireguard/wireguard.ui
  vpn/wireguard/wireguardadvanced.ui
  vpn/wireguard/wireguardadvancedwidget.cpp
  vpn/wireguard/wireguardadvancedwidget.h
  vpn/wireguard/wireguardauth.cpp
  vpn/wireguard/wireguardauth.h
  vpn/wireguard/wireguardauth.ui
  vpn/wireguard/wireguardkeyvalidator.cpp
  vpn/wireguard/wireguardkeyvalidator.h
  vpn/wireguard/wireguardwidget.cpp
  vpn/wireguard/wireguardwidget.h

To: andersonbruce, #plasma, jgrulich, pino
Cc: acrouthamel, K900, pino, lbeltrame, ngraham, plasma-devel, ragreen, Pitel, 
ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D15738: [Fonts KCM] remove filler words from anti-aliasing settings' labels

[Sven Mauch]

svenmauch added a comment.


  Looks good to me +1

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D15738

To: ngraham, #vdg, #plasma
Cc: svenmauch, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D15616: [Comic] Handle error state correctly

[Anthony Fieroni]

anthonyfieroni updated this revision to Diff 42286.

REPOSITORY
  R114 Plasma Addons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D15616?vs=42148&id=42286

REVISION DETAIL
  https://phabricator.kde.org/D15616

AFFECTED FILES
  applets/comic/comic.cpp
  applets/comic/comic.h
  dataengines/comic/comicprovider.cpp

To: anthonyfieroni, davidedmundson, #plasma
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart