D26531: [Wallpaper] Show author as subtitle in configuration

[Kai Uwe Broulik]

broulik added a comment.


  > can you look into that?
  
  Perhaps @ngraham can.

REPOSITORY
  R120 Plasma Workspace

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

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

Re: Long press global shortcuts

[Kai Uwe Broulik]

Hi,

I noticed on my phone all buttons (power, volume, home) react only on 
release.


On the desktop I don't know how annoying it would be to react only on 
press. However, that would also make it consistent with in-application 
shortcuts, which also trigger on press.


Perhaps it's best to only do on release triggers for shortcuts that 
actually have a long-press trigger?


Cheers
Kai Uwe

D26506: [KDED KCM] Rewrite as KDeclarative ScrollViewKCM

[Kai Uwe Broulik]

broulik updated this revision to Diff 73369.
broulik edited the summary of this revision.
broulik edited the test plan for this revision.
broulik added a comment.


  - Remove enable/disable animation
  - Enabled becomes green
  - Now monitors kded running
  - Notifies when modules got automatically started/stopped when settings were 
saved

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D26506?vs=73089=73369

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

AFFECTED FILES
  CMakeLists.txt
  kcms/kded/CMakeLists.txt
  kcms/kded/filterproxymodel.cpp
  kcms/kded/filterproxymodel.h
  kcms/kded/kcmkded.cpp
  kcms/kded/kcmkded.desktop
  kcms/kded/kcmkded.h
  kcms/kded/modulesmodel.cpp
  kcms/kded/modulesmodel.h
  kcms/kded/package/contents/ui/main.qml
  kcms/kded/package/metadata.desktop

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

D26586: [SystemTray] Rework system tray settings

[Nathaniel Graham]

ngraham added a comment.


  In D26586#592707 , @kmaterka wrote:
  
  > In D26586#592050 , @ngraham 
wrote:
  >
  > > Much better! The scrollview needs a frame around it though. You can do 
this by adding this to it:
  >
  >
  > Is it decided (D26530 )? Correct me if 
I'm wrong, in Kirigami scrollbars are overlaying with transparent background, 
Kirigami just adds some paddings when needed. Anyway, to have scrollbar 
seprated it is better to just add one margin to the list (and remove 
"rightPadding" I added to header, section and list item).
  
  
  The discussion about overlay vs non-overlay scrollbars is unrelated to 
whether or not to show a frame and background behind a scrollview. :) Just add 
the background for now I think.

REPOSITORY
  R120 Plasma Workspace

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

To: kmaterka, #plasma_workspaces, #plasma, davidedmundson, broulik, ngraham
Cc: meven, nicolasfella, plasma-devel, ngraham, LeGast00n, The-Feren-OS-Dev, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D22382: Add global shortcuts for switching to the previous/next activity

[Nathaniel Graham]

ngraham added a comment.


  Gross, I hate it when Phabricator does that. :( I still haven't figured out 
what causes this. Whenever it happens, I just go back in the history (in this 
case 
https://phabricator.kde.org/D22382?vs=on=61521=ignore-most#toc) 
copy the diff, manually apply it to HEAD and force-update the revision with 
`arc diff --update D22382`.
  
  If that doesn't work then I use the Update Diff action in the web interface 
to manually replace the current diff with a real one.
  
  Sorry about this nonsense. :/

REPOSITORY
  R120 Plasma Workspace

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

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

D22382: Add global shortcuts for switching to the previous/next activity

[Christian Muehlhaeuser]

muesli added a comment.


  Oops, I'm not sure what I've did here was what I intended: I rebased master 
(>800 commits since my original changeset) into my feature branch because I 
otherwise couldn't build/test my changes anymore, but obviously I didn't 
touch/change > 600 files.
  Is this how phabricator is supposed to look in such a scenario, or how should 
I have resolved this scenario?

REPOSITORY
  R120 Plasma Workspace

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

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

[plasmashell] [Bug 368942] Splash screen spinner should disappear instead of pause when fading into desktop

[Nate Graham]

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

Nate Graham  changed:

   What|Removed |Added

  Component|Global Theme packages   |Theme - Breeze

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

D26586: [SystemTray] Rework system tray settings

[Konrad Materka]

kmaterka added a comment.


  In D26586#592050 , @ngraham wrote:
  
  > Much better! The scrollview needs a frame around it though. You can do this 
by adding this to it:
  
  
  Is it decided (D26530 )? Correct me if 
I'm wrong, in Kirigami scrollbars are overlaying with transparent background, 
Kirigami just adds some paddings when needed. Anyway, to have scrollbar 
seprated it is better to just add one margin to the list (and remove 
"rightPadding" I added to header, section and list item).
  
  In D26586#592052 , @ngraham wrote:
  
  > Also for toggling individual sections, I would recommend making the button 
say "disable all" or "hide all" (depending on what's most accurate) and use the 
`view-hidden` icon. Then when all items in that category are disabled/hidden, 
disable all controls and labels for all items in that section.
  
  
  OK, will change the icon. I will disable all items. Or I'll just remove this 
feature...
  
  > Finally, if it's possible to use alternating stripey background colors 
instead of lines between items, that's generally preferred for multi-column 
table-style views.
  
  In plasmoidviewer all are white, I don't know why it is not the case in real 
usage. I will fix that.
  
  In D26586#592057 , @nicolasfella 
wrote:
  
  > Also German locale strikes again: The shortcut label (Tastatur-Kurzbefehle) 
is too long here
  
  
  OK, I will fix that. My idea is to get max size of all items in the column 
and apply it to all elements (like in real table).
  
  In D26586#592116 , @nicolasfella 
wrote:
  
  > I'm not against keeping the separators, but the button to en/disable the 
whole section seems overkill to me. Chances are that the set of entries you 
want to have does not exactly align with the categories anyway
  
  
  It is not that it is the first feature in KDE that almost no one uses :) But 
I lean to the idea of removing it. It really does not make much sense to hide 
whole category, because it applies to all apps - even those you don't know yet. 
Definitely, it is confusing. For me, as a develop, it is hard to remove 
features, it break my heart :) From the other side, I understand that is is 
equally important to add features and to remove useless ones. Do we have any 
vote against/to keep this feature?
  Note to myself: to remove this, you need to remove all usage of configuration 
properties: `cfg_*Shown`.

REPOSITORY
  R120 Plasma Workspace

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

To: kmaterka, #plasma_workspaces, #plasma, davidedmundson, broulik, ngraham
Cc: meven, nicolasfella, plasma-devel, ngraham, LeGast00n, The-Feren-OS-Dev, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D22382: Add global shortcuts for switching to the previous/next activity

[Christian Muehlhaeuser]

muesli updated this revision to Diff 73354.
muesli added a comment.


  - Remove default key-sequence for "Previous/Next Activity" shortcut

REPOSITORY
  R120 Plasma Workspace

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D22382?vs=61521=73354

BRANCH
  shortcut-prevnext-activity (branched from master)

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

AFFECTED FILES
  .gitignore
  CMakeLists.txt
  applets/CMakeLists.txt
  applets/activitybar/metadata.desktop
  applets/analog-clock/metadata.desktop
  applets/appmenu/lib/appmenuapplet.cpp
  applets/appmenu/package/contents/ui/main.qml
  applets/appmenu/package/metadata.desktop
  applets/appmenu/plugin/appmenumodel.cpp
  applets/batterymonitor/package/contents/ui/BatteryItem.qml
  applets/batterymonitor/package/contents/ui/BrightnessItem.qml
  applets/batterymonitor/package/contents/ui/CompactRepresentation.qml
  applets/batterymonitor/package/contents/ui/PopupDialog.qml
  applets/batterymonitor/package/contents/ui/batterymonitor.qml
  applets/batterymonitor/package/contents/ui/logic.js
  applets/batterymonitor/package/metadata.desktop
  applets/calendar/package/metadata.desktop
  applets/clipboard/contents/ui/ClipboardPage.qml
  applets/clipboard/contents/ui/ImageItemDelegate.qml
  applets/clipboard/metadata.desktop
  applets/devicenotifier/package/contents/ui/devicenotifier.qml
  applets/devicenotifier/package/metadata.desktop
  applets/devicenotifier/test-predicate-openinwindow.desktop
  applets/digital-clock/package/contents/config/config.qml
  applets/digital-clock/package/contents/ui/CalendarView.qml
  applets/digital-clock/package/contents/ui/DigitalClock.qml
  applets/digital-clock/package/contents/ui/configAppearance.qml
  applets/digital-clock/package/contents/ui/configTimeZones.qml
  applets/digital-clock/package/contents/ui/main.qml
  applets/digital-clock/package/metadata.desktop
  applets/digital-clock/plugin/CMakeLists.txt
  applets/digital-clock/plugin/clipboardmenu.cpp
  applets/icon/iconapplet.cpp
  applets/icon/package/metadata.desktop
  applets/kicker/CMakeLists.txt
  applets/kicker/Messages.sh
  applets/kicker/plugin/abstractentry.cpp
  applets/kicker/plugin/abstractentry.h
  applets/kicker/plugin/abstractmodel.cpp
  applets/kicker/plugin/abstractmodel.h
  applets/kicker/plugin/actionlist.cpp
  applets/kicker/plugin/actionlist.h
  applets/kicker/plugin/appentry.cpp
  applets/kicker/plugin/appentry.h
  applets/kicker/plugin/appsmodel.cpp
  applets/kicker/plugin/appsmodel.h
  applets/kicker/plugin/computermodel.cpp
  applets/kicker/plugin/computermodel.h
  applets/kicker/plugin/contactentry.cpp
  applets/kicker/plugin/contactentry.h
  applets/kicker/plugin/containmentinterface.cpp
  applets/kicker/plugin/containmentinterface.h
  applets/kicker/plugin/dashboardwindow.cpp
  applets/kicker/plugin/dashboardwindow.h
  applets/kicker/plugin/draghelper.cpp
  applets/kicker/plugin/draghelper.h
  applets/kicker/plugin/fileentry.cpp
  applets/kicker/plugin/fileentry.h
  applets/kicker/plugin/forwardingmodel.cpp
  applets/kicker/plugin/forwardingmodel.h
  applets/kicker/plugin/funnelmodel.cpp
  applets/kicker/plugin/funnelmodel.h
  applets/kicker/plugin/kastatsfavoritesmodel.cpp
  applets/kicker/plugin/kastatsfavoritesmodel.h
  applets/kicker/plugin/kickerplugin.cpp
  applets/kicker/plugin/kickerplugin.h
  applets/kicker/plugin/menuentryeditor.cpp
  applets/kicker/plugin/menuentryeditor.h
  applets/kicker/plugin/placeholdermodel.cpp
  applets/kicker/plugin/placeholdermodel.h
  applets/kicker/plugin/processrunner.cpp
  applets/kicker/plugin/processrunner.h
  applets/kicker/plugin/qmldir
  applets/kicker/plugin/recentcontactsmodel.cpp
  applets/kicker/plugin/recentcontactsmodel.h
  applets/kicker/plugin/recentusagemodel.cpp
  applets/kicker/plugin/recentusagemodel.h
  applets/kicker/plugin/rootmodel.cpp
  applets/kicker/plugin/rootmodel.h
  applets/kicker/plugin/runnermatchesmodel.cpp
  applets/kicker/plugin/runnermatchesmodel.h
  applets/kicker/plugin/runnermodel.cpp
  applets/kicker/plugin/runnermodel.h
  applets/kicker/plugin/simplefavoritesmodel.cpp
  applets/kicker/plugin/simplefavoritesmodel.h
  applets/kicker/plugin/submenu.cpp
  applets/kicker/plugin/submenu.h
  applets/kicker/plugin/systementry.cpp
  applets/kicker/plugin/systementry.h
  applets/kicker/plugin/systemmodel.cpp
  applets/kicker/plugin/systemmodel.h
  applets/kicker/plugin/systemsettings.cpp
  applets/kicker/plugin/systemsettings.h
  applets/kicker/plugin/wheelinterceptor.cpp
  applets/kicker/plugin/wheelinterceptor.h
  applets/kicker/plugin/windowsystem.cpp
  applets/kicker/plugin/windowsystem.h
  applets/lock_logout/contents/config/config.qml
  applets/lock_logout/metadata.desktop
  applets/mediacontroller/contents/ui/main.qml
  applets/mediacontroller/metadata.desktop
  applets/notifications/CMakeLists.txt
  applets/notifications/globalshortcuts.cpp
  applets/notifications/globalshortcuts.h
  applets/notifications/notificationapplet.cpp
  

D26609: [kstyle] Port to KWindowSystem shadows API

[Vlad Zahorodnii]

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

REPOSITORY
  R113 Oxygen Theme

BRANCH
  port-to-shadows-api

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

AFFECTED FILES
  kstyle/oxygenshadowhelper.cpp
  kstyle/oxygenshadowhelper.h

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

D26458: [windowsystem] Provide a KWindowShadow implementation

[Vlad Zahorodnii]

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


  Track Shadow object

REPOSITORY
  R130 Frameworks integration plugin using KWayland

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D26458?vs=72863=73350

BRANCH
  port-to-shadows-api

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

AFFECTED FILES
  src/windowsystem/CMakeLists.txt
  src/windowsystem/plugin.cpp
  src/windowsystem/plugin.h
  src/windowsystem/waylandintegration.cpp
  src/windowsystem/waylandintegration.h
  src/windowsystem/windowshadow.cpp
  src/windowsystem/windowshadow.h

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

D26475: [kstyle] Port to KWindowSystem shadows API

[Vlad Zahorodnii]

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


  Re-write the entire port :/
  
  In case of server-side drop-shadows, we don't want to destroy KWindowShadow
  before QPlatformSurfaceEvent::SurfaceAboutToBeDestroyed is emitted, otherwise
  the associated window will be shadow-less when it's being animated by the
  compositor.
  
  In general, we need to clean up KWindowShadow internals when QWindow sends
  QPlatformSurfaceEvent::SurfaceDestroyed or something, but unfortunately Qt
  has no such event. :(
  
  I would say that this use-case is pretty weird (I mean the intentional
  resource leak). On the other hand, having release() or something that says
  "I want to leak resource blah-blah" is also weird.
  
  It would be great to have the following event filter
  
bool ShadowHelper::eventFilter(QObject *watched, QEvent *event)
{
if (event->type() == QEvent::PlatformSurface) {
QWidget *widget = static_cast(watched);
QPlatformSurfaceEvent *surfaceEvent = 
static_cast(event);
switch (surfaceEvent->surfaceEventType()) {
case QPlatformSurfaceEvent::SurfaceCreated:
installShadows(widget);
break;
case QPlatformSurfaceEvent::SurfaceAboutToBeDestroyed:
// Don't care.
break;
case QPlatformSurfaceEvent::SurfaceDestroyed:
uninstallShadows(widget); // will do nothing but cleanup
break;
}
}
return false;
}

REPOSITORY
  R31 Breeze

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D26475?vs=72924=73349

BRANCH
  port-to-shadows-api

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

AFFECTED FILES
  kstyle/breezeshadowhelper.cpp
  kstyle/breezeshadowhelper.h

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

Long press global shortcuts

[Aleix Pol]

Hi,
In a discussion some weeks ago, we came to the conclusion that it
would make sense to support long press global shortcuts.

The use-case we discussed back then was the power button on the Plasma
phone. We need to have 2 actions upon press. Short press would lock
and dim the screen, long press would show the shutdown button.
Thinking about it further makes me think it's something we probably
want for the desktop use-case as well.

The problem is two-fold:
- We need to support long-press on kglobal accel (or even QShortcut?)
- When do we trigger the action? Because now it happens on pressed.

The second part of this is why I'm sending this e-mail. We're
responding to global shortcuts on press rather than on release, which
makes it impossible to handle the long press. Would it make sense to
change it to on release? Maybe only in the cases where there's a long
press counterpart?

Aleix

D25947: Add applet for controlling Night Color

[Yuri Chornoivan]

yurchor added a comment.


  In D25947#592565 , @victorr wrote:
  
  > There is a small typo.
  >  You need to replace the quotation marks.
  >  Attached a patch.
  >  F7879213: kdeplasma-addons-5.17.80-fix.patch 

  
  
  Fixed.
  
https://commits.kde.org/kdeplasma-addons/e76ed74f056bdbbe69c2242b75aa77624f7e471b

REPOSITORY
  R114 Plasma Addons

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

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

D26584: Load localization catalog for slideshow config window

[David Edmundson]

davidedmundson added a comment.


  > I'm still not totally clear on why some things require a domain to be added 
for translations, and other things don't.
  
  Everything requires a domain set in some way.
  
  For apps/C++ this is done effectively magically at compile time.
  
  For QML we have to have some code somewhere that says "this QML context 
should use this i18n domain", which is all in kdeclarative. This is already 
quite messy, but the real complication is when one context loads a plugin from 
something else - we haven't solved that, so we specify a domain explicitly.

REPOSITORY
  R120 Plasma Workspace

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

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

D25947: Add applet for controlling Night Color

[Victor Ryzhykh]

victorr added a comment.


  There is a small typo.
  You need to replace the quotation marks.
  Attached a patch.
  F7879213: kdeplasma-addons-5.17.80-fix.patch 


REPOSITORY
  R114 Plasma Addons

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

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

D26582: Kicker/RecentUsageModel: Allow to open smb:/ sftp:/ resources

[Méven Car]

meven added a comment.


  In D26582#592266 , @aleksejshilin 
wrote:
  
  > In D26582#591959 , @meven wrote:
  >
  > > Well there was already KRun::setRunExecutables :)
  >
  >
  > I've already talked about it in my previous message:
  >
  > In D26582#591952 , 
@aleksejshilin wrote:
  >
  > > KRun currently doesn't seem to have a way to force opening of executables 
- one can only prohibit running them with setRunExecutables(), in which case an 
error message is displayed. That's why I decided to go the way it is now.
  >
  >
  > For example:
  >
  > 1. Install Okteta.
  > 2. Associate application/x-sharedlib mime type with Okteta.
  > 3. Open a shared library with Okteta.
  > 4. Try to open it again from 'Recent Documents'.
  >
  >   Instead of opening the file in Okteta, it'll show a message box saying 
that the executable file won't be started for security reasons.
  >
  >   I admit that it's not a likely scenario though, so let's see what others 
have to say.
  
  
  Thanks for testing this. There is a different treatment with nativeBinary in 
KRun,
  We may want to change KRun::runUrl when a native binary is to be run but with 
runExecutable is false, around line 400:
  
} else if (isNativeBinary) {
// Show warning for executables that aren't scripts
noRun = true;
}
  
  I guess in this case with could check for mime-type associated apps like 
oketa can be with any binary file, perhaps adding a flag.

REPOSITORY
  R120 Plasma Workspace

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

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

D26584: Load localization catalog for slideshow config window

[Yuri Chornoivan]

This revision was automatically updated to reflect the committed changes.
Closed by commit R120:025387c931b3: Load localization catalog for slideshow 
config window (authored by yurchor).

REPOSITORY
  R120 Plasma Workspace

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D26584?vs=73279=73331

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

AFFECTED FILES
  wallpapers/image/imagepackage/contents/ui/config.qml

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