Re: Plasma 6 alpha modules list

[Ismael Asensio]

Hi Jonathan.

The repo ocean-sound-theme needs to be included into the plasma release
repos.

Sorry if I overlooked it.

El vie., 27 oct. 2023 17:39, Jonathan Riddell  escribió:

> 12 days to go until the first alpha release, it's time to work out some
> details.
>
> set(PROJECT_VERSION_MAJOR 5) shall I change to 6 now? This is used for
> soversions for various internal libs
>
> We've probably discussed but I've forgotted, what minimum version of Qt?
> Currently 6.4 is set for most but do we want 6.6?
>
> When is feature freeze?  Before Alpha? Or should we call that soft feature
> freeze and have a harder one end of November before the Beta?
>
> plasma-bigscreen - No Qt 6 port, drop?
>
> aura-browser - bigscreen app, Qt 6 port but drop?
> plank-player - bigscreen app, Qt 6 port but drop?
> plasma-remotecontrollers - bigscreen plugin, Qt 6 port good but drop?
>
> kwrited - can we get rid of this? I remmeber having great fun at uni
> writing onto people's terminals but it feels like the write/mesg concept is
> something that should die with old school unix.  I got it to work in KDE
> neon using xterm after I edited /etc/login.defs but not with konsole or
> gnome-terminal
>
> plasma-mobile - good although it would be nice to get confirmation from
> the plasma mobile team they want it released at the same time
>
> Niccolo says he wants gamepad-kcm in which i think is
> https://invent.kde.org/redstrate/gamepad-kcm that needs kdereviewed
> pronto (or just put into plasma-workspace?)
>
> bluedevil - good
> breeze - Qt 5 and 6 in one for qt style theme
> breeze-grub - good, no change needed
> breeze-gtk - "find_package(Breeze 5.14.90 REQUIRED)" should this be
> updated to 5.27.80 (it builds and installs fine with Qt 5 or 6 for now)
> breeze-plymouth - good, no Qt needed
> discover - good, needs new unreleased libappstream
> drkonqi - good
> flatpak-kcm - good
> kactivitymanagerd - good
> kde-cli-tools - good
> kde-gtk-config - good
> kdecoration - good
> kdeplasma-addons - good
> kgamma5 - good but can we drop the 5 from the repo name?
> khotkeys - drop
> kinfocenter - good
> kmenuedit - good
> kpipewire - good
> kscreen - good
> kscreenlocker - good
> ksshaskpass - good
> ksystemstats - good
> kwallet-pam - good
> kwayland-integration - drop
> kwin - good
> layer-shell-qt - good
> libkscreen - good
> libksysguard - good
> milou - good
> oxygen - good, builds Qt 5 and 6 in one
> oxygen-sounds - good (no Qt)
> plasma-browser-integration - good
> plasma-desktop - good
> plasma-disks - good
> plasma-firewall - good
> plasma-integration - good
> plasma-nano - good (but does it have any users?)
> plasma-nm - good
> plasma-pa - good
> plasma-sdk - good but can I suggest rename cuttlefish to iconexplorer or
> something more explanatory?
> plasma-systemmonitor - good
> plasma-thunderbolt - good
> plasma-vault - good
> plasma-welcome - good but change REQUIRED_QT_VERSION to QT_MIN_VERSION ?
> plasma-workspace - good
> plasma-workspace-wallpapers - good
> plymouth-kcm - good but QT_MIN_VERSION is still 5 so I'll bump it
> polkit-kde-agent-1 - good
> powerdevil - good
> qqc2-breeze-style - good for Qt 6 but I think we need to release a Qt 5
> build using the Plasma/5.27 branch but using a different name (neon uses
> qqc2-breeze5-style)
> sddm-kcm - good
> systemsettings - good
> xdg-desktop-portal-kde - good
>
>
>

Re: plasma 5.25 tars

[Ismael Asensio]

Hi Jonathan!

Would you mind for a respin of plasma-desktop to include
2138f96d17c1db351c348e8501a0e2a516414832?

Without this, there seemed to be problems, at least on some Fedora fresh
installations, but it slipped our minds till last minute.
Sorry about that

El jue, 9 jun 2022 a las 15:52, Jonathan Riddell ()
escribió:

> source tars are up for packaging now for plasma 5.25.0.  release due
> tuesday
>
>
>

D29395: Refactor MediaController

[Ismael Asensio]

iasensio added a comment.


  Any chance to retake this and move it to invent?
  
  This is really useful for an extension of the media controller I'm 
mantaining, and will help to eventually upsream it (al least partially)
  I have a branch based on this MR with some fixes to it, and collaboration in 
gitlab seems easier

REPOSITORY
  R120 Plasma Workspace

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

To: cblack, #plasma
Cc: meven, kmaterka, iasensio, broulik, trmdi, ngraham, plasma-devel, Orage, 
LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, 
ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, 
apol, ahiemstra, mart

D26002: [KCMs/Night Color] Fix 1-digit minutes representation

[Ismael Asensio]

This revision was automatically updated to reflect the committed changes.
Closed by commit R119:275195373cf1: [KCMs/Night Color] Fix 1-digit minutes 
representation (authored by iasensio).

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D26002?vs=71549=71554

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

AFFECTED FILES
  kcms/nightcolor/package/contents/ui/TimingsView.qml

To: iasensio, #plasma, #vdg, davidedmundson
Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, 
ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D26002: [nightcolor] Fix 1-digit minutes representation

[Ismael Asensio]

iasensio created this revision.
iasensio added reviewers: Plasma, VDG.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
iasensio requested review of this revision.

REVISION SUMMARY
  On automatic time modes, minutes 0-9 were represented only with 1-digit
  (i.e 8:0 instead of 8:00)

TEST PLAN
  Example Lat: 40.4 Lon: -3.7 
  (or travel to a place where sunrise starts at 8:00AM)

REPOSITORY
  R119 Plasma Desktop

BRANCH
  fix_pretty_time

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

AFFECTED FILES
  kcms/nightcolor/package/contents/ui/TimingsView.qml

To: iasensio, #plasma, #vdg
Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, 
ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D24953: [applets/weather] Use unit description from KUnitConversion

[Ismael Asensio]

iasensio updated this revision to Diff 68853.
iasensio added a comment.


  - Rebase

REPOSITORY
  R114 Plasma Addons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D24953?vs=68771=68853

BRANCH
  weather_use_unit_description

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

AFFECTED FILES
  applets/weather/plugin/abstractunitlistmodel.h
  applets/weather/plugin/plugin.cpp
  applets/weather/plugin/util.cpp
  applets/weather/plugin/util.h

To: iasensio, #plasma, ngraham
Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, 
ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D24964: [applets/weather] Port from KIconLoader to QIcon

[Ismael Asensio]

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

REVISION SUMMARY
  `KIconLoader` was used only to check if the `iconName` is valid.
  This patch uses `QIcon::hasThemeIcon()` instead.
  It might help T11637  a little.

TEST PLAN
  No behavior changes

REPOSITORY
  R114 Plasma Addons

BRANCH
  weather_qicon

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

AFFECTED FILES
  applets/weather/plugin/util.cpp

To: iasensio, #plasma
Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, 
ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D24953: [applets/weather] Use unit description from KUnitConversion

[Ismael Asensio]

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

REVISION SUMMARY
  Use unit description directly from KUnitConversion instead of making a new 
string.
  This avoids duplicating translation strings and improves consistency.
  The units name is now `description` (`symbol`) also for consistency with 
krunner

TEST PLAN
  No behavior changes within the applet
  `Configure` -> `Units` shows translated `description` (`symbol`) for every 
unit

REPOSITORY
  R114 Plasma Addons

BRANCH
  weather_use_unit_description

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

AFFECTED FILES
  applets/weather/plugin/abstractunitlistmodel.h
  applets/weather/plugin/plugin.cpp
  applets/weather/plugin/util.cpp
  applets/weather/plugin/util.h

To: iasensio, #plasma
Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, 
ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D24952: Add mmHg to pressure options in weather applet

[Ismael Asensio]

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

REVISION SUMMARY
  Add millimeters of mercury (mmHg) to pressure options in weather applet, 
which is a common pressure unit in some countries
  
  FEATURE: 401734
  FIXED-IN: 5.18

TEST PLAN
  You can select 'mmHg' as the pressure unit and it displays correctly in the 
widget

REPOSITORY
  R114 Plasma Addons

BRANCH
  weather_mmHg

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

AFFECTED FILES
  applets/weather/plugin/plugin.cpp

To: iasensio, #plasma
Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, 
ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D23064: Fix duplicate results when a 2nd unit is partially written in krunner

[Ismael Asensio]

iasensio added a comment.


  Sure about the email. Thanks!

REPOSITORY
  R114 Plasma Addons

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

To: iasensio, ngraham, broulik
Cc: plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, 
ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, 
apol, mart

D23064: Fix #410744: Duplicate results when a 2nd unit is partially written in krunner

[Ismael Asensio]

iasensio created this revision.
iasensio added reviewers: ngraham, broulik.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
iasensio requested review of this revision.

REVISION SUMMARY
  When typing a conversion, and the second unit is partially written (i.e. "10€ 
to ru..." the results are duplicated by several times. This behavior happens 
with different categories of units (lenght, currency, ...) and on different 
languages.
  
  The problem is that a new result is being added not once for every matching 
unit, but once for every matching stringUnit, generating the duplicates.
  
  Before:
  F7202568: krunner_before.png 
  
  After:
  F7202571: krunner_after.png 
  
  This is my first attempt to build kde from source and commit the diff to 
phabricator, so please excuse any possible mistakes.
  
  BUG: 410744

TEST PLAN
  Open krunner and type any conversion with a letter for the second unit: "10€ 
to r"
  Result for each matching second unit is only shown one time

REPOSITORY
  R114 Plasma Addons

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

AFFECTED FILES
  runners/converter/converterrunner.cpp

To: iasensio, ngraham, broulik
Cc: plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, 
ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, 
apol, mart