[plasmashell] [Bug 350691] plasma crash if system notification pop up

2015-07-27 Thread Kai Uwe Broulik
https://bugs.kde.org/show_bug.cgi?id=350691

Kai Uwe Broulik  changed:

   What|Removed |Added

Product|kwrited |plasmashell
  Component|general |Notifications
   Assignee|plasma-devel@kde.org|mklape...@kde.org
 CC||plasma-b...@kde.org
   Target Milestone|--- |1.0

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[kwrited] [Bug 350691] plasma crash if system notification pop up

2015-07-27 Thread Kai Uwe Broulik
https://bugs.kde.org/show_bug.cgi?id=350691

Kai Uwe Broulik  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEEDSINFO
 CC||k...@privat.broulik.de
 Resolution|--- |WAITINGFORINFO

--- Comment #1 from Kai Uwe Broulik  ---
Can you try to get a backtrace, please?

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[kwrited] [Bug 350691] New: plasma crash if system notification pop up

2015-07-27 Thread nicky
https://bugs.kde.org/show_bug.cgi?id=350691

Bug ID: 350691
   Summary: plasma crash if system notification pop up
   Product: kwrited
   Version: 5.3.2
  Platform: Archlinux Packages
OS: Linux
Status: UNCONFIRMED
  Severity: normal
  Priority: NOR
 Component: general
  Assignee: plasma-devel@kde.org
  Reporter: rogers619952...@gmail.com

Every time when my laptop has low power or can't connect to network, kde
notification will pop up without any information and then all plasmas will
freeze while other applications still run well. I use Archlinux with Nvidia
card and installed the driver(nouveau) for my card. 

Reproducible: Always

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124453: [digital-clock] Provide a simple option for 24h clock

2015-07-27 Thread Thomas Pfeiffer


> On July 26, 2015, 12:35 a.m., Thomas Pfeiffer wrote:
> > I fully agree with you, in both that this should only be a temporary 
> > workaround, and on that it is very helpful as such.
> > +1 from me!
> 
> Martin Klapetek wrote:
> What's your take on Kai's comment about using a combobox for the options? 
> Currently it is a single checkbox, see the screenshot at [1]. I think you 
> either want to use 24h or you want to use 12h clock and I think that one does 
> not change locales that offten to warrant a "system default", a combo imho is 
> then a needless clutter to the settings dialog. Comments?
> 
> [1] 
> https://lh5.googleusercontent.com/-riysQFXKRfs/VbJqhhB8yBI/T4U/Zy205nGVXF0/w904-h574-no/digital-clock-5_4.png

I agree with Martin: Locale isn't likely to change so often, so I don't think 
it's much of an annoyance to have to adapt the setting to your liking each time 
you change the locale.
I'd default it the the one the locale chooses and then allow users to change it 
manually, period.


- Thomas


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124453/#review82967
---


On July 25, 2015, 9:55 p.m., Martin Klapetek wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124453/
> ---
> 
> (Updated July 25, 2015, 9:55 p.m.)
> 
> 
> Review request for Plasma and Thomas Pfeiffer.
> 
> 
> Bugs: 345378
> https://bugs.kde.org/show_bug.cgi?id=345378
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> ---
> 
> This is _the_ most requested feature for the clock, simple checkbox for 24h 
> clock.
> 
> In my opinion this is just a temporary workaround and not a very good one 
> (generally,
> code-wise it's awesome :P). There should really be a global config option for 
> that
> but as we now rely on QLocale and QLocale is...bad at this, here's at least 
> this
> small help, which will at least change the clock format on the panel.
> 
> Imo we could really really use something like KLocale again.
> 
> 
> Diffs
> -
> 
>   applets/digital-clock/package/contents/config/main.xml 8b0872b 
>   applets/digital-clock/package/contents/ui/DigitalClock.qml 9506078 
>   applets/digital-clock/package/contents/ui/configAppearance.qml ead0ad5 
> 
> Diff: https://git.reviewboard.kde.org/r/124453/diff/
> 
> 
> Testing
> ---
> 
> Switching the checkbox on and off works as expected.
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Powerdevil] [Bug 350686] Configure screen locking

2015-07-27 Thread Kai Uwe Broulik
https://bugs.kde.org/show_bug.cgi?id=350686

Kai Uwe Broulik  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |WORKSFORME
 CC||k...@privat.broulik.de

--- Comment #1 from Kai Uwe Broulik  ---
The setting can be found in System Settings -> Workspace Behaviour -> Lock
Screen

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Powerdevil] [Bug 350686] New: Configure screen locking

2015-07-27 Thread LtWorf
https://bugs.kde.org/show_bug.cgi?id=350686

Bug ID: 350686
   Summary: Configure screen locking
   Product: Powerdevil
   Version: 5.3.2
  Platform: Debian unstable
OS: Linux
Status: UNCONFIRMED
  Severity: normal
  Priority: NOR
 Component: general
  Assignee: plasma-devel@kde.org
  Reporter: tipos...@tiscali.it

On my machine, the screen locks automatically after a short period of
inactivity.

While this is a neat security feature, it is a bit pointless for those who live
alone. However there does not seem to be any configuration for the screen
locking.

Reproducible: Always

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124499: Detect initial face icon being created

2015-07-27 Thread Aleix Pol Gonzalez

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124499/#review83065
---



src/qmlcontrols/kcoreaddons/kuserproxy.cpp (line 47)


Is the watch always needed or only if it's empty?

This wasn't added before...


- Aleix Pol Gonzalez


On July 27, 2015, 11:32 p.m., David Edmundson wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124499/
> ---
> 
> (Updated July 27, 2015, 11:32 p.m.)
> 
> 
> Review request for Plasma and Harald Sitter.
> 
> 
> Repository: kdeclarative
> 
> 
> Description
> ---
> 
> Detect initial face icon being created.
> 
> I think it's a bit messy, but can't come up with anything else. 
> Potentially will be replaced by AccountsService soon anyway...
> 
> 
> Diffs
> -
> 
>   src/qmlcontrols/kcoreaddons/kuserproxy.cpp 
> 383220fc901010cc2b7d9a5b7f7fc3928835b445 
> 
> Diff: https://git.reviewboard.kde.org/r/124499/diff/
> 
> 
> Testing
> ---
> 
> Literally none.
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Review Request 124499: Detect initial face icon being created

2015-07-27 Thread David Edmundson

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124499/
---

Review request for Plasma and Harald Sitter.


Repository: kdeclarative


Description
---

Detect initial face icon being created.

I think it's a bit messy, but can't come up with anything else. 
Potentially will be replaced by AccountsService soon anyway...


Diffs
-

  src/qmlcontrols/kcoreaddons/kuserproxy.cpp 
383220fc901010cc2b7d9a5b7f7fc3928835b445 

Diff: https://git.reviewboard.kde.org/r/124499/diff/


Testing
---

Literally none.


Thanks,

David Edmundson

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Breeze] [Bug 350684] New: lock window when switching between users is very hard to use

2015-07-27 Thread Lauris
https://bugs.kde.org/show_bug.cgi?id=350684

Bug ID: 350684
   Summary: lock window when switching between users is very hard
to use
   Product: Breeze
   Version: 5.3.1
  Platform: Ubuntu Packages
OS: Linux
Status: UNCONFIRMED
  Severity: minor
  Priority: NOR
 Component: general
  Assignee: plasma-devel@kde.org
  Reporter: lau...@nix.lv

When switching between users or creating new user session in lock screen it
would be much easier to use it if it would after clicking "Switch" or "Create"
it would go to main screen where you with focus on password. Also would be nice
if when dblclicked on icon it would also work just like when you would have
clicked button "Switch" button.

Reproducible: Always

Steps to Reproduce:
1. Select user
2. Click "Switch"


Actual Results:  
Nothing happens (at least visually). You have to click "Cancel" to get back to
unlock screen and only than you see that user has actually switched.

Expected Results:  
After clicking "Switch" or double clicking on User icon it would switch to user
session and switch to main screen with focus on password input box.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124255: keyboard: do not reset shortcut for changing layout

2015-07-27 Thread Ivan Romanov

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124255/
---

(Updated July 27, 2015, 9:23 p.m.)


Status
--

This change has been marked as submitted.


Review request for Plasma.


Changes
---

Submitted with commit fcdd29ac411a6806200c51887d64620f15dee998 by Ivan Romanov 
to branch master.


Repository: plasma-desktop


Description
---

This patch solves a problem when Ctrl-Alt-K always used for Switch to Next 
Keyboard Layout. It solves this bug [RHBZ 
1234082](https://bugzilla.redhat.com/show_bug.cgi?id=1234082)


Diffs
-

  kcms/keyboard/bindings.cpp a11941f 

Diff: https://git.reviewboard.kde.org/r/124255/diff/


Testing
---


Thanks,

Ivan Romanov

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Review Request 124495: Proofread ksysguard docbook for kf5

2015-07-27 Thread Burkhard Lück

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124495/
---

Review request for Documentation and Plasma.


Repository: ksysguard


Description
---

Proofread + replaced entity kde with plasma
Remove launching System Activities from krunner


Diffs
-

  doc/index.docbook 7a65b62 

Diff: https://git.reviewboard.kde.org/r/124495/diff/


Testing
---

builds


Thanks,

Burkhard Lück

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Powerdevil] [Bug 350676] Need to press key twice to decrease brightness

2015-07-27 Thread Kai Uwe Broulik
https://bugs.kde.org/show_bug.cgi?id=350676

Kai Uwe Broulik  changed:

   What|Removed |Added

 CC||k...@privat.broulik.de
 Ever confirmed|0   |1
 Status|UNCONFIRMED |CONFIRMED

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Powerdevil] [Bug 350676] New: Need to press key twice to decrease brightness

2015-07-27 Thread Daniel Vrátil
https://bugs.kde.org/show_bug.cgi?id=350676

Bug ID: 350676
   Summary: Need to press key twice to decrease brightness
   Product: Powerdevil
   Version: 5.3.90
  Platform: Other
OS: Linux
Status: UNCONFIRMED
  Severity: normal
  Priority: NOR
 Component: general
  Assignee: plasma-devel@kde.org
  Reporter: dvra...@kde.org

Changing brightness via keys (Fn+F5/Fn+F6) works, but I need to press the key
twice in order to decrease the brightness. Increasing brightness works as
expected.

HW: Thinkpad T540p
PowerManagement -> PowerDevil -> Maximum Screen Brightness says 4794.

Let me know if you need any additional data.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Notes from Plasma BoF

2015-07-27 Thread Sebastian Kügler
On Monday, July 27, 2015 15:43:12 Vishesh Handa wrote:
> On Mon, Jul 27, 2015 at 3:01 PM, Sebastian Kügler  wrote:
> > - new Xapian-replacement backend for Baloo (check with Vishesh)
> 
> Yup.
> 
> Also Baloo is moving to Frameworks, and will not be part of Plasma 5.4

Any scheduling issues here (because, been there, done that)?

Have you already checked if the baloo-related repos you're moving will be in 
Frameworks in time for the next Plasma release?
-- 
sebas

Sebastian Kügler|http://vizZzion.org| http://kde.org

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124296: Update autostart kcm docbook to kf5

2015-07-27 Thread Burkhard Lück

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124296/
---

(Updated Juli 27, 2015, 5:12 nachm.)


Review request for Documentation, Plasma and Jeremy Whiting.


Changes
---

Fix issues spotted by Luigi, use &plasma; entity


Repository: plasma-desktop


Description
---

I found a lot of usefull infos for the update in this blog post, thanks a lot 
Jeremy
http://jpwhiting.blogspot.de/2014/11/autostart-in-plasma-5.html

Some questions:

1)
.config/plaswa-workspace/env -> pre kde startup (immediately after login)
.config/plaswa-workspace/shutdown -> shutdown
.config/autostart/ startup only for programs (Desktop files) 
using/launching scripts after kde startup is not possible?
how to enable/disable a script? not possible afaik
Is this correct?
How to deal with the issue that it is no longer possible to launch a script at 
startup?
Any fix in sight or should I add a note about that missing feature to the 
docbook?

2) 
migration from kde4:
Copy Desktop files from $HOME/.kde/Autostart to $HOME/.config/autostart/
Copy Script files to from $HOME/.kde/Autostart to 
$HOME/.config/plasma-workspace/env (pre kde startup) 
or $HOME/.config/plasma-workspace/shutdown (shutdown scripts)
Should I add a note about migration from kde4 to the docbook?

3)
Use Plasma instead of &kde;?
GUI catalog kcm_autostart still has several messages with KDE: 
"Autostart only in KDE" "KDE Autostart Manager" "KDE Autostart Manager Control 
Panel Module" "Pre-KDE startup"
I'd prefer to use Plasma instead of &kde; in the docbook, but then the GUI has 
to be changed as well

4) 
Disabling Autostart Files Provided By Your Distribution -> 
http://www.sharpley.org.uk/node/8
Add a note about this to the docbook?

5)
KDE Applications handling autostart on their own e.g.
KAlarm enable/disable autostart in settings dialog
Konversation/Kopete in systemtray, you have to use File->Quit to prevent 
autostart on next login
Add a note about this to the docbook?

6)
Background Services - Startup Services:
This shows all KDE services that can be loaded on KDE startup. Checked services 
will be invoked on next startup.
Add a note about this or link to the relevant kcm docbook?

7)
Desktop Session - another way to autostart applications
Add a note about this or link to the relevant kcm docbook?


Diffs (updated)
-

  doc/kcontrol/autostart/index.docbook c16f8a3 

Diff: https://git.reviewboard.kde.org/r/124296/diff/


Testing
---

builds


Thanks,

Burkhard Lück

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 123468: Add setting to adjust screen scaling

2015-07-27 Thread Kai Uwe Broulik


> On Juli 27, 2015, 10:36 vorm., Daniel Vrátil wrote:
> > Another idea: should we somehow somewhere show a message that this only 
> > applies after logout/login? Otherwise we get bugreports that it does not 
> > work :)

Yeah perhaps place a KMessageWidget at the top that says that (or the ugly 
dialog we usually use in fonts)


- Kai Uwe


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123468/#review83031
---


On Juli 26, 2015, 2:35 nachm., David Edmundson wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123468/
> ---
> 
> (Updated Juli 26, 2015, 2:35 nachm.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: kscreen
> 
> 
> Description
> ---
> 
> Single dialog alters both QT_DEVICE_PIXEL_RAITO and XRDB.DPI text
> scaling factor that used to reside in fonts KCM.
> 
> A preview widget shows how this will look on the that monitor.
> 
> Changes take affect after logout/login; not ideal but we're limited by
> the Qt QPA for now. Will try and fix that after.
> 
> edit: having uploaded this I can see I have some whitespace left, please 
> don't feel the need to tell me.
> 
> 
> Diffs
> -
> 
>   CMakeLists.txt e157a5e28e441a2e6dacb2d639cf6d120fb18c26 
>   kcm/src/CMakeLists.txt 50bfdf037c331fe7c6763fb85c3b43857cbea5d5 
>   kcm/src/previewwidget.h PRE-CREATION 
>   kcm/src/previewwidget.cpp PRE-CREATION 
>   kcm/src/scaling.ui PRE-CREATION 
>   kcm/src/scalingconfig.h PRE-CREATION 
>   kcm/src/scalingconfig.cpp PRE-CREATION 
>   kcm/src/stylepreview.ui PRE-CREATION 
>   kcm/src/widget.cpp 7fe96c1c8b21b19424ef4993dff9eb3985bcefdb 
> 
> Diff: https://git.reviewboard.kde.org/r/123468/diff/
> 
> 
> Testing
> ---
> 
> 
> File Attachments
> 
> 
> the dialog
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2015/04/22/a8cab37c-24bf-4fb9-b50b-39bc34596938__snapshot1.png
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 122580: Fixed visibility of play/pause button

2015-07-27 Thread Abhimanyu Tak

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122580/
---

(Updated July 27, 2015, 1:55 p.m.)


Status
--

This change has been discarded.


Review request for Plasma and Bhushan Shah.


Repository: plasma-mediacenter


Description
---

The play/pause and stop button in MediaController is now visible and is 
functioning properly.


Diffs
-

  mediaelements/mediacontroller/MediaController.qml 5947a68 

Diff: https://git.reviewboard.kde.org/r/122580/diff/


Testing
---


Thanks,

Abhimanyu Tak

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Notes from Plasma BoF

2015-07-27 Thread Vishesh Handa
On Mon, Jul 27, 2015 at 3:01 PM, Sebastian Kügler  wrote:
> - new Xapian-replacement backend for Baloo (check with Vishesh)

Yup.

Also Baloo is moving to Frameworks, and will not be part of Plasma 5.4


Vishesh Handa
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Notes from Plasma BoF

2015-07-27 Thread Bhushan Shah
On Mon, Jul 27, 2015 at 6:31 PM, Sebastian Kügler  wrote:
>
> - Plasma 5.5 planning
> - khotkeys: remove gestures from khotkeys ui

I take that, in fact I have patch in stash or somewhere which needs
some fixing and way to RB.

> - Formfactor filtering
> - make sure that plasmoids have formfactor (sebas)
> - filter in systemtray
> - filter formfactor in Kickoff
> - filter in add widget dialog

+1, Will be happy to help.. I have similar use-case for PMC.

-- 
Bhushan Shah

http://bhush9.github.io
IRC Nick : bshah on Freenode

On Mon, Jul 27, 2015 at 6:31 PM, Sebastian Kügler  wrote:
> Plasma BoF 27-7-2015
>
> - Qt 5.5
> - Plasma 5.4 does not depend on 5.5, but we support / fix bugs with Qt 5.5
> - we recommend Qt 5.4
> - seems all happy, but let's look out for bugs
>
>
> - Changelog for Plasma 5.4
> - we recommend Qt 5.5, multiscreen is much better with it
> - improved high dpi support (should complete high dpi support)
> - to merge Dave's slider
> - touch improvements (in task manager for example)
> - memory improvements (script engine stuff)
> - Easier clock setup for 24 hour mode
> - calendar navigation improvements
> - plasma-volume-control
> - new Xapian-replacement backend for Baloo (check with Vishesh)
> - sticky notes with drag and drop support, keyboard nav
> - trash applet now works again with drag and drop
> - system tray has more options (show all, faster and more efficient, more
> reliable)
> - improved wallpaper rendering
> - Wayland tech preview (complete Plasma wayland session), driven by phone
> feature set
> - plus stuff that comes up on the ML
> - shit-ton more breeze icons
> - improved documentation
> - new wallpaper
>
>
> - plasma-volume-control
> - needs final review / moving to kde/workspace -> poke Harald
> - make sure shortcuts are working (Harald)
> - ship .desktop file to not autostart replace kmix (Dave)
> - check i18n (sebas)
>
> - high dpi: we want to investigate dpi validation heuristics
>
> - Plasma 5.5 planning
> - holiday and PIM integration
> - review / merge phone repos into kde/workspace (only workspacey parts,
> other things should go into frameworks or apps)
> - merge davetray (turns xembed systray stuff into statusnotifiers, makes
> it work also on wayland)
> - more granularity for kauthorized / kiosk checks
> - Wayland improvements:
> - screen configuration for wayland changes
> - kwin QPA
> - get moving of windows fixed
> - popup positions
> - move screenlocker into kwin
> - hard-depend on libwayland
> - touchpad KCM rewrite
> - killing consolekit usage
> - kill HAL usage (barring *BSD objections)
> - formfactor filtering (see phone stuff)
> - evaluate and adopt phabricator
> - virtualbox support
> - khotkeys: remove gestures from khotkeys ui
>
> - Phone stuff
> - look Rob Clark's (vanilla) kernels instead of the Android / libhybris
> stack
> - look at steps to have ubuntu touch supported devices work
> - Discuss moving to Debian as reference platform
> - untangle ubuntu touch apps support
> - new libhybris
> - Does Kubuntu / Neon want to base on Debian
> - plan Plasma sprint for September (sebas)
> - more documentation
> - issue migration to bugzilla (sebas)
> - Phone design BoF
> - bigger root filesystem
> - easy setup to run phone shell on the desktop (add that to the SDK)
> - Formfactor filtering
> - make sure that plasmoids have formfactor (sebas)
> - filter in systemtray
> - filter formfactor in Kickoff
> - filter in add widget dialog
> - how to do applications / HIG for phones
>
> - priority fix bugs for Plasma 5.4
> - shortcuts are broken, needs fixing (dave) #307390 (some key combos not
> working as shortcuts); #348456 (custom shortcuts not sent)
> - CPU usage in plotter component #348385 and #346134
> - systray to activate individual entries #350610 (probably too invasive
> for 5.4 -> push to 5.5)
> - solid crashed plasmashell #346517
> - keyboard shortcuts and i18n clas #309193
>
>
>
>
> --
> sebas
>
> Sebastian Kügler|http://vizZzion.org| http://kde.org
>
> ___
> Plasma-devel mailing list
> Plasma-devel@kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel



-- 
Bhushan Shah

http://bhush9.github.io
IRC Nick : bshah on Freenode
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 122488: Improved calendar navigation

2015-07-27 Thread Sebastian Kügler


> On July 27, 2015, 9:55 a.m., John Layt wrote:
> > Fantastic to see this :-) Pretty much as I documented it at 
> > https://community.kde.org/Plasma/Clock#Zooming_Calendar (which was a 
> > serious crib from Windows anyway :-) ). The thing to add for the future 
> > will be clicking on the day takes you to an Agenda view with your calendar 
> > and holidays and even weather forecast for that day :-)

That's within reach with KDEPIM joining the applications in 15.08. We've put 
that features on the plan for Plasma 5.5.


- Sebastian


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122488/#review83024
---


On July 27, 2015, 9:43 a.m., Kai Uwe Broulik wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122488/
> ---
> 
> (Updated July 27, 2015, 9:43 a.m.)
> 
> 
> Review request for Plasma and KDE Usability.
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> ---
> 
> This improves the calendar navigation by providing a "Year overview" showing 
> all 12 months in a grid, and a "Decade overview" showing the current decade 
> in a grid.
> 
> A lot of code has just been moved around. The overviews use a QML ListModel 
> owing to laziness.
> 
> See https://www.youtube.com/watch?v=7SaBhRa32ds for a screencast (I love that 
> mouse click effect!)
> 
> 
> Diffs
> -
> 
>   src/declarativeimports/calendar/qml/MonthView.qml 601755f 
>   src/declarativeimports/calendar/qml/DaysCalendar.qml ab3e750 
>   src/declarativeimports/calendar/qml/DayDelegate.qml 6a3747e 
>   src/declarativeimports/calendar/daysmodel.cpp 1a6f454 
>   src/declarativeimports/calendar/daysmodel.h e1285f6 
>   src/declarativeimports/calendar/daydata.h 39ac086 
>   src/declarativeimports/calendar/calendar.h 5dc3081 
>   src/declarativeimports/calendar/calendar.cpp c462dbd 
> 
> Diff: https://git.reviewboard.kde.org/r/122488/diff/
> 
> 
> Testing
> ---
> 
> I changed the selection to be persistent during navigation; other than that, 
> should work as before, with the new overviews.
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124255: keyboard: do not reset shortcut for changing layout

2015-07-27 Thread Martin Klapetek

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124255/#review83046
---

Ship it!


Looks good to me

- Martin Klapetek


On July 6, 2015, 11:21 a.m., Ivan Romanov wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124255/
> ---
> 
> (Updated July 6, 2015, 11:21 a.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> ---
> 
> This patch solves a problem when Ctrl-Alt-K always used for Switch to Next 
> Keyboard Layout. It solves this bug [RHBZ 
> 1234082](https://bugzilla.redhat.com/show_bug.cgi?id=1234082)
> 
> 
> Diffs
> -
> 
>   kcms/keyboard/bindings.cpp a11941f 
> 
> Diff: https://git.reviewboard.kde.org/r/124255/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Ivan Romanov
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124296: Update autostart kcm docbook to kf5

2015-07-27 Thread Luigi Toscano

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124296/#review82866
---



doc/kcontrol/autostart/index.docbook (line 16)


Plasma (we should really use entities)



doc/kcontrol/autostart/index.docbook (line 38)


currently



doc/kcontrol/autostart/index.docbook (line 51)


I would follow the new patterns and talk about "KDE Workspaces 4" "KDE 
Plasma Workspaces 4" or "KDE Plasma 1" or "previous version of KDE Plasma" 
instead of "&kde; 4".



doc/kcontrol/autostart/index.docbook (line 103)


.config


- Luigi Toscano


On Lug. 23, 2015, 10:09 p.m., Burkhard Lück wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124296/
> ---
> 
> (Updated Lug. 23, 2015, 10:09 p.m.)
> 
> 
> Review request for Documentation, Plasma and Jeremy Whiting.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> ---
> 
> I found a lot of usefull infos for the update in this blog post, thanks a lot 
> Jeremy
> http://jpwhiting.blogspot.de/2014/11/autostart-in-plasma-5.html
> 
> Some questions:
> 
> 1)
> .config/plaswa-workspace/env -> pre kde startup (immediately after login)
> .config/plaswa-workspace/shutdown -> shutdown
> .config/autostart/ startup only for programs (Desktop files) 
> using/launching scripts after kde startup is not possible?
> how to enable/disable a script? not possible afaik
> Is this correct?
> How to deal with the issue that it is no longer possible to launch a script 
> at startup?
> Any fix in sight or should I add a note about that missing feature to the 
> docbook?
> 
> 2) 
> migration from kde4:
> Copy Desktop files from $HOME/.kde/Autostart to $HOME/.config/autostart/
> Copy Script files to from $HOME/.kde/Autostart to 
> $HOME/.config/plasma-workspace/env (pre kde startup) 
> or $HOME/.config/plasma-workspace/shutdown (shutdown scripts)
> Should I add a note about migration from kde4 to the docbook?
> 
> 3)
> Use Plasma instead of &kde;?
> GUI catalog kcm_autostart still has several messages with KDE: 
> "Autostart only in KDE" "KDE Autostart Manager" "KDE Autostart Manager 
> Control Panel Module" "Pre-KDE startup"
> I'd prefer to use Plasma instead of &kde; in the docbook, but then the GUI 
> has to be changed as well
> 
> 4) 
> Disabling Autostart Files Provided By Your Distribution -> 
> http://www.sharpley.org.uk/node/8
> Add a note about this to the docbook?
> 
> 5)
> KDE Applications handling autostart on their own e.g.
> KAlarm enable/disable autostart in settings dialog
> Konversation/Kopete in systemtray, you have to use File->Quit to prevent 
> autostart on next login
> Add a note about this to the docbook?
> 
> 6)
> Background Services - Startup Services:
> This shows all KDE services that can be loaded on KDE startup. Checked 
> services will be invoked on next startup.
> Add a note about this or link to the relevant kcm docbook?
> 
> 7)
> Desktop Session - another way to autostart applications
> Add a note about this or link to the relevant kcm docbook?
> 
> 
> Diffs
> -
> 
>   doc/kcontrol/autostart/index.docbook c16f8a3 
> 
> Diff: https://git.reviewboard.kde.org/r/124296/diff/
> 
> 
> Testing
> ---
> 
> builds
> 
> 
> Thanks,
> 
> Burkhard Lück
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Notes from Plasma BoF

2015-07-27 Thread Sebastian Kügler
On Monday, July 27, 2015 15:19:16 Martin Klapetek wrote:
> On Mon, Jul 27, 2015 at 3:01 PM, Sebastian Kügler  wrote:
> Plasma BoF 27-7-2015

Brainfart.

> - Qt 5.5
> - we recommend Qt 5.4

This is wrong.

> - Changelog for Plasma 5.4
> - we recommend Qt 5.5, multiscreen is much better with it

This is right.

> These^ two seem contradictory?
-- 
sebas

Sebastian Kügler|http://vizZzion.org| http://kde.org

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Plasma mobile and Fairphone

2015-07-27 Thread Sebastian Kügler
On Monday, July 27, 2015 11:55:50 Martin Steigerwald wrote:
> > Let's coordinate before acting, please.
> 
> Sebastian, I acted before I saw the mails from Mario and you. As from Alex 
> response "we lack the people who could approach them" I thought there may
> not  be anyone else doing it, so I just acted.
> 
> I am sorry in case you think this was not in your interest and I happily
> let  it to Mario and you to contact Fairphone. I have holidays anyway, so I
> even appreciate if someone else moves it forward.

Aight, no problem then. I just want us to look like a coordinated team with a 
common direction and not like a headless chicken. Didn't mean to sound harsh.

Cheers, and thanks for the good intention!
-- 
sebas

Sebastian Kügler|http://vizZzion.org| http://kde.org

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Notes from Plasma BoF

2015-07-27 Thread Martin Klapetek
On Mon, Jul 27, 2015 at 3:01 PM, Sebastian Kügler  wrote:

> Plasma BoF 27-7-2015
>
> - Qt 5.5
> - we recommend Qt 5.4
>
> - Changelog for Plasma 5.4
> - we recommend Qt 5.5, multiscreen is much better with it
>

These^ two seem contradictory?

Cheers
-- 
Martin Klapetek | KDE Developer
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124485: Add missing traffic monitor

2015-07-27 Thread Jan Grulich


> On Čec. 27, 2015, 12:57 odp., Lamarque Souza wrote:
> > applet/contents/ui/ConnectionItem.qml, line 306
> > 
> >
> > what's the update rate used here? I think it is still 1 second. The old 
> > Plasma NM used to use 2 seconds instead. Increasing the update time can 
> > also help with the update overhead.

Also 2 seconds.


> On Čec. 27, 2015, 12:57 odp., Lamarque Souza wrote:
> > applet/contents/ui/ConnectionItem.qml, line 346
> > 
> >
> > I do not understand why returning when source name is found. Sorry for 
> > not looking at DataSource documentation for the answer but my laptop is 
> > malfunctioning. I am doing this review from my tablet.

Hmm, I don't remember right now, but I guess it checks whether the sourceName 
starts with "network/interfaces/DeviceName".


> On Čec. 27, 2015, 12:57 odp., Lamarque Souza wrote:
> > applet/contents/ui/ConnectionItem.qml, line 519
> > 
> >
> > so at least write the original file path you copied that from.

Ok, will do that.


- Jan


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124485/#review83041
---


On Čec. 27, 2015, 1:08 odp., Jan Grulich wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124485/
> ---
> 
> (Updated Čec. 27, 2015, 1:08 odp.)
> 
> 
> Review request for Network Management, Plasma and KDE Usability.
> 
> 
> Bugs: 333070
> http://bugs.kde.org/show_bug.cgi?id=333070
> 
> 
> Repository: plasma-nm
> 
> 
> Description
> ---
> 
> See summary.
> 
> 
> Diffs
> -
> 
>   applet/contents/ui/ConnectionItem.qml ff722a7 
> 
> Diff: https://git.reviewboard.kde.org/r/124485/diff/
> 
> 
> Testing
> ---
> 
> ![Traffic monitor](https://jgrulich.fedorapeople.org/plotter.png)
> 
> 
> Thanks,
> 
> Jan Grulich
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124485: Add missing traffic monitor

2015-07-27 Thread Jan Grulich

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124485/
---

(Updated July 27, 2015, 1:08 p.m.)


Status
--

This change has been marked as submitted.


Review request for Network Management, Plasma and KDE Usability.


Changes
---

Submitted with commit e20839806450cd018de2e7a24fb23a54a70fe6da by Jan Grulich 
to branch master.


Bugs: 333070
http://bugs.kde.org/show_bug.cgi?id=333070


Repository: plasma-nm


Description
---

See summary.


Diffs
-

  applet/contents/ui/ConnectionItem.qml ff722a7 

Diff: https://git.reviewboard.kde.org/r/124485/diff/


Testing
---

![Traffic monitor](https://jgrulich.fedorapeople.org/plotter.png)


Thanks,

Jan Grulich

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Notes from Plasma BoF

2015-07-27 Thread David Edmundson
On Mon, Jul 27, 2015 at 3:01 PM, Sebastian Kügler  wrote:

> Plasma BoF 27-7-2015
>
> - Qt 5.5
> - Plasma 5.4 does not depend on 5.5, but we support / fix bugs with Qt
> 5.5
> - we recommend Qt 5.4
> - seems all happy, but let's look out for bugs
>
>
> - Changelog for Plasma 5.4
> - we recommend Qt 5.5, multiscreen is much better with it
> - improved high dpi support (should complete high dpi support)
> - to merge Dave's slider
> - touch improvements (in task manager for example)
> - memory improvements (script engine stuff)
> - Easier clock setup for 24 hour mode
> - calendar navigation improvements
> - plasma-volume-control
> - new Xapian-replacement backend for Baloo (check with Vishesh)
> - sticky notes with drag and drop support, keyboard nav
> - trash applet now works again with drag and drop
> - system tray has more options (show all, faster and more efficient,
> more
> reliable)
> - improved wallpaper rendering
> - Wayland tech preview (complete Plasma wayland session), driven by
> phone
> feature set
> - plus stuff that comes up on the ML
> - shit-ton more breeze icons
> - improved documentation
> - new wallpaper
>
>
> - plasma-volume-control
> - needs final review / moving to kde/workspace -> poke Harald
> - make sure shortcuts are working (Harald)
> - ship .desktop file to not autostart replace kmix (Dave)
> - check i18n (sebas)
>
> - high dpi: we want to investigate dpi validation heuristics
>
> - Plasma 5.5 planning
> - holiday and PIM integration
> - review / merge phone repos into kde/workspace (only workspacey parts,
> other things should go into frameworks or apps)
> - merge davetray (turns xembed systray stuff into statusnotifiers,
> makes
> it work also on wayland)
> - more granularity for kauthorized / kiosk checks
> - Wayland improvements:
> - screen configuration for wayland changes
> - kwin QPA
> - get moving of windows fixed
> - popup positions
> - move screenlocker into kwin
> - hard-depend on libwayland
> - touchpad KCM rewrite
> - killing consolekit usage
> - kill HAL usage (barring *BSD objections)
> - formfactor filtering (see phone stuff)
> - evaluate and adopt phabricator
> - virtualbox support
> - khotkeys: remove gestures from khotkeys ui
>
> - Phone stuff
> - look Rob Clark's (vanilla) kernels instead of the Android / libhybris
> stack
> - look at steps to have ubuntu touch supported devices work
> - Discuss moving to Debian as reference platform
> - untangle ubuntu touch apps support
> - new libhybris
> - Does Kubuntu / Neon want to base on Debian
> - plan Plasma sprint for September (sebas)
> - more documentation
> - issue migration to bugzilla (sebas)
> - Phone design BoF
> - bigger root filesystem
> - easy setup to run phone shell on the desktop (add that to the SDK)
> - Formfactor filtering
> - make sure that plasmoids have formfactor (sebas)
> - filter in systemtray
> - filter formfactor in Kickoff
> - filter in add widget dialog
> - how to do applications / HIG for phones
>
> - priority fix bugs for Plasma 5.4
> - shortcuts are broken, needs fixing (dave) #307390 (some key combos
> not
> working as shortcuts); #348456 (custom shortcuts not sent)
>


> - CPU usage in plotter component #348385 and #346134
>
Fixed now :)


> - systray to activate individual entries #350610 (probably too invasive
> for 5.4 -> push to 5.5)
>


> - solid crashed plasmashell #346517
>

Can someone volunteer to look into this one?


> - keyboard shortcuts and i18n clas #309193
>
> and this one

>
>
>
> --
> sebas
>
> Sebastian Kügler|http://vizZzion.org| http://kde.org
>
> ___
> Plasma-devel mailing list
> Plasma-devel@kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel
>
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Notes from Plasma BoF

2015-07-27 Thread Sebastian Kügler
Plasma BoF 27-7-2015

- Qt 5.5
- Plasma 5.4 does not depend on 5.5, but we support / fix bugs with Qt 5.5
- we recommend Qt 5.4
- seems all happy, but let's look out for bugs


- Changelog for Plasma 5.4
- we recommend Qt 5.5, multiscreen is much better with it
- improved high dpi support (should complete high dpi support)
- to merge Dave's slider
- touch improvements (in task manager for example)
- memory improvements (script engine stuff)
- Easier clock setup for 24 hour mode
- calendar navigation improvements
- plasma-volume-control
- new Xapian-replacement backend for Baloo (check with Vishesh)
- sticky notes with drag and drop support, keyboard nav
- trash applet now works again with drag and drop
- system tray has more options (show all, faster and more efficient, more 
reliable)
- improved wallpaper rendering
- Wayland tech preview (complete Plasma wayland session), driven by phone 
feature set
- plus stuff that comes up on the ML
- shit-ton more breeze icons
- improved documentation
- new wallpaper


- plasma-volume-control
- needs final review / moving to kde/workspace -> poke Harald
- make sure shortcuts are working (Harald)
- ship .desktop file to not autostart replace kmix (Dave)
- check i18n (sebas)

- high dpi: we want to investigate dpi validation heuristics

- Plasma 5.5 planning
- holiday and PIM integration
- review / merge phone repos into kde/workspace (only workspacey parts, 
other things should go into frameworks or apps)
- merge davetray (turns xembed systray stuff into statusnotifiers, makes 
it work also on wayland)
- more granularity for kauthorized / kiosk checks
- Wayland improvements:
- screen configuration for wayland changes
- kwin QPA
- get moving of windows fixed
- popup positions
- move screenlocker into kwin
- hard-depend on libwayland
- touchpad KCM rewrite
- killing consolekit usage
- kill HAL usage (barring *BSD objections)
- formfactor filtering (see phone stuff)
- evaluate and adopt phabricator
- virtualbox support
- khotkeys: remove gestures from khotkeys ui

- Phone stuff
- look Rob Clark's (vanilla) kernels instead of the Android / libhybris 
stack
- look at steps to have ubuntu touch supported devices work
- Discuss moving to Debian as reference platform
- untangle ubuntu touch apps support
- new libhybris
- Does Kubuntu / Neon want to base on Debian
- plan Plasma sprint for September (sebas)
- more documentation
- issue migration to bugzilla (sebas)
- Phone design BoF
- bigger root filesystem
- easy setup to run phone shell on the desktop (add that to the SDK)
- Formfactor filtering
- make sure that plasmoids have formfactor (sebas)
- filter in systemtray
- filter formfactor in Kickoff
- filter in add widget dialog
- how to do applications / HIG for phones

- priority fix bugs for Plasma 5.4
- shortcuts are broken, needs fixing (dave) #307390 (some key combos not 
working as shortcuts); #348456 (custom shortcuts not sent)
- CPU usage in plotter component #348385 and #346134
- systray to activate individual entries #350610 (probably too invasive 
for 5.4 -> push to 5.5)
- solid crashed plasmashell #346517
- keyboard shortcuts and i18n clas #309193




-- 
sebas

Sebastian Kügler|http://vizZzion.org| http://kde.org

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124485: Add missing traffic monitor

2015-07-27 Thread Lamarque Souza

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124485/#review83041
---



applet/contents/ui/ConnectionItem.qml (line 306)


what's the update rate used here? I think it is still 1 second. The old 
Plasma NM used to use 2 seconds instead. Increasing the update time can also 
help with the update overhead.



applet/contents/ui/ConnectionItem.qml (line 346)


I do not understand why returning when source name is found. Sorry for not 
looking at DataSource documentation for the answer but my laptop is 
malfunctioning. I am doing this review from my tablet.



applet/contents/ui/ConnectionItem.qml (line 519)


so at least write the original file path you copied that from.


- Lamarque Souza


On July 27, 2015, 10:55 a.m., Jan Grulich wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124485/
> ---
> 
> (Updated July 27, 2015, 10:55 a.m.)
> 
> 
> Review request for Network Management, Plasma and KDE Usability.
> 
> 
> Bugs: 333070
> http://bugs.kde.org/show_bug.cgi?id=333070
> 
> 
> Repository: plasma-nm
> 
> 
> Description
> ---
> 
> See summary.
> 
> 
> Diffs
> -
> 
>   applet/contents/ui/ConnectionItem.qml ff722a7 
> 
> Diff: https://git.reviewboard.kde.org/r/124485/diff/
> 
> 
> Testing
> ---
> 
> ![Traffic monitor](https://jgrulich.fedorapeople.org/plotter.png)
> 
> 
> Thanks,
> 
> Jan Grulich
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Breeze] [Bug 347123] OSD is distracting and annoying

2015-07-27 Thread Martin Klapetek
https://bugs.kde.org/show_bug.cgi?id=347123

--- Comment #13 from Martin Klapetek  ---
Plasma 5.4 will bring much improved OSD, besides it getting smaller, the
disappearing algorithm was also changed to be less "annoying".

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Breeze] [Bug 347123] OSD is distracting and annoying

2015-07-27 Thread Martin Klapetek
https://bugs.kde.org/show_bug.cgi?id=347123

Martin Klapetek  changed:

   What|Removed |Added

   Version Fixed In||5.4.0
 Resolution|--- |FIXED
  Latest Commit||http://commits.kde.org/plas
   ||ma-workspace/3956cac47d2a15
   ||184b0ecb4ca84049b2ab6b1870
 Status|UNCONFIRMED |RESOLVED

--- Comment #12 from Martin Klapetek  ---
Git commit 3956cac47d2a15184b0ecb4ca84049b2ab6b1870 by Martin Klapetek.
Committed on 27/07/2015 at 11:57.
Pushed by mklapetek into branch 'master'.

[osd] Make the osd 1/6th of a screen at most
FIXED-IN: 5.4.0

M  +1-1lookandfeel/contents/osd/OsdItem.qml

http://commits.kde.org/plasma-workspace/3956cac47d2a15184b0ecb4ca84049b2ab6b1870

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124451: [digital-clock] Add support for ISO date format

2015-07-27 Thread Martin Klapetek

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124451/
---

(Updated July 27, 2015, 1:45 p.m.)


Review request for Plasma.


Repository: plasma-workspace


Description
---

Another frequently requested feature - allow date to be displayed
in ISO format (-MM-DD)


Diffs (updated)
-

  applets/digital-clock/package/contents/ui/DigitalClock.qml 9506078 
  applets/digital-clock/package/contents/ui/configAppearance.qml ead0ad5 
  applets/digital-clock/package/contents/ui/main.qml f42d7ad 

Diff: https://git.reviewboard.kde.org/r/124451/diff/


Testing
---

ISO date appears as expected.


Thanks,

Martin Klapetek

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124485: Add missing traffic monitor

2015-07-27 Thread David Edmundson

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124485/#review83040
---

Ship it!


Ship It!

- David Edmundson


On July 27, 2015, 10:55 a.m., Jan Grulich wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124485/
> ---
> 
> (Updated July 27, 2015, 10:55 a.m.)
> 
> 
> Review request for Network Management, Plasma and KDE Usability.
> 
> 
> Bugs: 333070
> http://bugs.kde.org/show_bug.cgi?id=333070
> 
> 
> Repository: plasma-nm
> 
> 
> Description
> ---
> 
> See summary.
> 
> 
> Diffs
> -
> 
>   applet/contents/ui/ConnectionItem.qml ff722a7 
> 
> Diff: https://git.reviewboard.kde.org/r/124485/diff/
> 
> 
> Testing
> ---
> 
> ![Traffic monitor](https://jgrulich.fedorapeople.org/plotter.png)
> 
> 
> Thanks,
> 
> Jan Grulich
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124485: Add missing traffic monitor

2015-07-27 Thread Jan Grulich


> On Čec. 27, 2015, 11:12 dop., David Edmundson wrote:
> > applet/contents/ui/ConnectionItem.qml, line 292
> > 
> >
> > I don't get why 5?

Because there are 5 lines in the plotter.


> On Čec. 27, 2015, 11:12 dop., David Edmundson wrote:
> > applet/contents/ui/ConnectionItem.qml, line 519
> > 
> >
> > can you document this?

Actually I can't, I just copied this part from the system monitor applet, but 
it just generates the second color used for upload.


- Jan


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124485/#review83030
---


On Čec. 27, 2015, 10:55 dop., Jan Grulich wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124485/
> ---
> 
> (Updated Čec. 27, 2015, 10:55 dop.)
> 
> 
> Review request for Network Management, Plasma and KDE Usability.
> 
> 
> Bugs: 333070
> http://bugs.kde.org/show_bug.cgi?id=333070
> 
> 
> Repository: plasma-nm
> 
> 
> Description
> ---
> 
> See summary.
> 
> 
> Diffs
> -
> 
>   applet/contents/ui/ConnectionItem.qml ff722a7 
> 
> Diff: https://git.reviewboard.kde.org/r/124485/diff/
> 
> 
> Testing
> ---
> 
> ![Traffic monitor](https://jgrulich.fedorapeople.org/plotter.png)
> 
> 
> Thanks,
> 
> Jan Grulich
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124485: Add missing traffic monitor

2015-07-27 Thread David Edmundson


> On July 27, 2015, 10:47 a.m., Heiko Tietze wrote:
> > Why are the graphs such unrealistic shaped? Looks like heavy filtering. If 
> > Thomas and Jens granted usability I will check the ship it here. But please 
> > discuss also the scaling method. When it is adopted to the current speed 
> > the actual graph is not really informative; if the y-axis' scaling is fix 
> > to the max. bandwidth it might be a problem in case of other bottlenecks. I 
> > suggest to (optionally) scale by the max measured value therefore.
> 
> Jan Grulich wrote:
> That's the way how it's implemented in KDeclarative framework, I don't 
> know if it's possible to change it and I don't have knowledge to do it 
> myself. It scales according to the maximal visible value of the graph.

there is a property to explcitly set a max for this purpose; but you need 
automatic scaling here I think as you have no idea what the max speed of the 
network is.


- David


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124485/#review83033
---


On July 27, 2015, 10:55 a.m., Jan Grulich wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124485/
> ---
> 
> (Updated July 27, 2015, 10:55 a.m.)
> 
> 
> Review request for Network Management, Plasma and KDE Usability.
> 
> 
> Bugs: 333070
> http://bugs.kde.org/show_bug.cgi?id=333070
> 
> 
> Repository: plasma-nm
> 
> 
> Description
> ---
> 
> See summary.
> 
> 
> Diffs
> -
> 
>   applet/contents/ui/ConnectionItem.qml ff722a7 
> 
> Diff: https://git.reviewboard.kde.org/r/124485/diff/
> 
> 
> Testing
> ---
> 
> ![Traffic monitor](https://jgrulich.fedorapeople.org/plotter.png)
> 
> 
> Thanks,
> 
> Jan Grulich
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124485: Add missing traffic monitor

2015-07-27 Thread David Edmundson

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124485/#review83030
---



applet/contents/ui/ConnectionItem.qml (line 292)


I don't get why 5?



applet/contents/ui/ConnectionItem.qml (line 519)


can you document this?


- David Edmundson


On July 27, 2015, 10:55 a.m., Jan Grulich wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124485/
> ---
> 
> (Updated July 27, 2015, 10:55 a.m.)
> 
> 
> Review request for Network Management, Plasma and KDE Usability.
> 
> 
> Bugs: 333070
> http://bugs.kde.org/show_bug.cgi?id=333070
> 
> 
> Repository: plasma-nm
> 
> 
> Description
> ---
> 
> See summary.
> 
> 
> Diffs
> -
> 
>   applet/contents/ui/ConnectionItem.qml ff722a7 
> 
> Diff: https://git.reviewboard.kde.org/r/124485/diff/
> 
> 
> Testing
> ---
> 
> ![Traffic monitor](https://jgrulich.fedorapeople.org/plotter.png)
> 
> 
> Thanks,
> 
> Jan Grulich
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124485: Add missing traffic monitor

2015-07-27 Thread Jan Grulich

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124485/
---

(Updated Čec. 27, 2015, 10:55 dop.)


Review request for Network Management, Plasma and KDE Usability.


Bugs: 333070
http://bugs.kde.org/show_bug.cgi?id=333070


Repository: plasma-nm


Description
---

See summary.


Diffs
-

  applet/contents/ui/ConnectionItem.qml ff722a7 

Diff: https://git.reviewboard.kde.org/r/124485/diff/


Testing
---

![Traffic monitor](https://jgrulich.fedorapeople.org/plotter.png)


Thanks,

Jan Grulich

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124485: Add missing traffic monitor

2015-07-27 Thread Jan Grulich


> On Čec. 27, 2015, 10:47 dop., Heiko Tietze wrote:
> > Why are the graphs such unrealistic shaped? Looks like heavy filtering. If 
> > Thomas and Jens granted usability I will check the ship it here. But please 
> > discuss also the scaling method. When it is adopted to the current speed 
> > the actual graph is not really informative; if the y-axis' scaling is fix 
> > to the max. bandwidth it might be a problem in case of other bottlenecks. I 
> > suggest to (optionally) scale by the max measured value therefore.

That's the way how it's implemented in KDeclarative framework, I don't know if 
it's possible to change it and I don't have knowledge to do it myself. It 
scales according to the maximal visible value of the graph.


- Jan


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124485/#review83033
---


On Čec. 27, 2015, 10:17 dop., Jan Grulich wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124485/
> ---
> 
> (Updated Čec. 27, 2015, 10:17 dop.)
> 
> 
> Review request for Network Management, Plasma and KDE Usability.
> 
> 
> Bugs: 268022
> http://bugs.kde.org/show_bug.cgi?id=268022
> 
> 
> Repository: plasma-nm
> 
> 
> Description
> ---
> 
> See summary.
> 
> 
> Diffs
> -
> 
>   applet/contents/ui/ConnectionItem.qml ff722a7 
> 
> Diff: https://git.reviewboard.kde.org/r/124485/diff/
> 
> 
> Testing
> ---
> 
> ![Traffic monitor](https://jgrulich.fedorapeople.org/plotter.png)
> 
> 
> Thanks,
> 
> Jan Grulich
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Plasma 5.4 Changelog

2015-07-27 Thread Aleix Pol
On Mon, Jul 27, 2015 at 10:23 AM, David Edmundson
 wrote:
> Plamsa freeze is Wednesday with the release (with dot article) on the 11th
> August
>
> Can people who made notable new features in Plasma please write a short
> summary of each thing they want in the release anouncement in this thread,
> so we know what to highlight/screenshot in the article.
>
> Thanks
>
> David
>
> ___
> Plasma-devel mailing list
> Plasma-devel@kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel
>

Maybe it would be worth looking at the initial plan, there might be
things done that we could forget:
https://notes.kde.org/p/plasma-5.4

To put them together, maybe it would also be good to use an etherpad:
https://notes.kde.org/p/plasma-5.4-changes

HTH,
Aleix
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124485: Add missing traffic monitor

2015-07-27 Thread Heiko Tietze

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124485/#review83033
---

Ship it!


Why are the graphs such unrealistic shaped? Looks like heavy filtering. If 
Thomas and Jens granted usability I will check the ship it here. But please 
discuss also the scaling method. When it is adopted to the current speed the 
actual graph is not really informative; if the y-axis' scaling is fix to the 
max. bandwidth it might be a problem in case of other bottlenecks. I suggest to 
(optionally) scale by the max measured value therefore.

- Heiko Tietze


On Juli 27, 2015, 10:17 vorm., Jan Grulich wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124485/
> ---
> 
> (Updated Juli 27, 2015, 10:17 vorm.)
> 
> 
> Review request for Network Management, Plasma and KDE Usability.
> 
> 
> Bugs: 268022
> http://bugs.kde.org/show_bug.cgi?id=268022
> 
> 
> Repository: plasma-nm
> 
> 
> Description
> ---
> 
> See summary.
> 
> 
> Diffs
> -
> 
>   applet/contents/ui/ConnectionItem.qml ff722a7 
> 
> Diff: https://git.reviewboard.kde.org/r/124485/diff/
> 
> 
> Testing
> ---
> 
> ![Traffic monitor](https://jgrulich.fedorapeople.org/plotter.png)
> 
> 
> Thanks,
> 
> Jan Grulich
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 123468: Add setting to adjust screen scaling

2015-07-27 Thread Àlex Fiestas

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123468/#review83032
---


Would be nice to adjust gtk stuff as well, same values should work in theory.

- Àlex Fiestas


On jul. 26, 2015, 2:35 p.m., David Edmundson wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123468/
> ---
> 
> (Updated jul. 26, 2015, 2:35 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: kscreen
> 
> 
> Description
> ---
> 
> Single dialog alters both QT_DEVICE_PIXEL_RAITO and XRDB.DPI text
> scaling factor that used to reside in fonts KCM.
> 
> A preview widget shows how this will look on the that monitor.
> 
> Changes take affect after logout/login; not ideal but we're limited by
> the Qt QPA for now. Will try and fix that after.
> 
> edit: having uploaded this I can see I have some whitespace left, please 
> don't feel the need to tell me.
> 
> 
> Diffs
> -
> 
>   CMakeLists.txt e157a5e28e441a2e6dacb2d639cf6d120fb18c26 
>   kcm/src/CMakeLists.txt 50bfdf037c331fe7c6763fb85c3b43857cbea5d5 
>   kcm/src/previewwidget.h PRE-CREATION 
>   kcm/src/previewwidget.cpp PRE-CREATION 
>   kcm/src/scaling.ui PRE-CREATION 
>   kcm/src/scalingconfig.h PRE-CREATION 
>   kcm/src/scalingconfig.cpp PRE-CREATION 
>   kcm/src/stylepreview.ui PRE-CREATION 
>   kcm/src/widget.cpp 7fe96c1c8b21b19424ef4993dff9eb3985bcefdb 
> 
> Diff: https://git.reviewboard.kde.org/r/123468/diff/
> 
> 
> Testing
> ---
> 
> 
> File Attachments
> 
> 
> the dialog
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2015/04/22/a8cab37c-24bf-4fb9-b50b-39bc34596938__snapshot1.png
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 123468: Add setting to adjust screen scaling

2015-07-27 Thread Daniel Vrátil

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123468/#review83031
---


Another idea: should we somehow somewhere show a message that this only applies 
after logout/login? Otherwise we get bugreports that it does not work :)

- Daniel Vrátil


On July 26, 2015, 4:35 p.m., David Edmundson wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123468/
> ---
> 
> (Updated July 26, 2015, 4:35 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: kscreen
> 
> 
> Description
> ---
> 
> Single dialog alters both QT_DEVICE_PIXEL_RAITO and XRDB.DPI text
> scaling factor that used to reside in fonts KCM.
> 
> A preview widget shows how this will look on the that monitor.
> 
> Changes take affect after logout/login; not ideal but we're limited by
> the Qt QPA for now. Will try and fix that after.
> 
> edit: having uploaded this I can see I have some whitespace left, please 
> don't feel the need to tell me.
> 
> 
> Diffs
> -
> 
>   CMakeLists.txt e157a5e28e441a2e6dacb2d639cf6d120fb18c26 
>   kcm/src/CMakeLists.txt 50bfdf037c331fe7c6763fb85c3b43857cbea5d5 
>   kcm/src/previewwidget.h PRE-CREATION 
>   kcm/src/previewwidget.cpp PRE-CREATION 
>   kcm/src/scaling.ui PRE-CREATION 
>   kcm/src/scalingconfig.h PRE-CREATION 
>   kcm/src/scalingconfig.cpp PRE-CREATION 
>   kcm/src/stylepreview.ui PRE-CREATION 
>   kcm/src/widget.cpp 7fe96c1c8b21b19424ef4993dff9eb3985bcefdb 
> 
> Diff: https://git.reviewboard.kde.org/r/123468/diff/
> 
> 
> Testing
> ---
> 
> 
> File Attachments
> 
> 
> the dialog
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2015/04/22/a8cab37c-24bf-4fb9-b50b-39bc34596938__snapshot1.png
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124433: [Panel] Delay disappearance of widget deletion dialog

2015-07-27 Thread David Kahles

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124433/
---

(Updated July 27, 2015, 12:30 p.m.)


Status
--

This change has been marked as submitted.


Review request for Plasma.


Repository: plasma-desktop


Description
---

Increase the delay to 360ms, and hide the dialog immediately if
a button on the dialog is clicked by the user.
If a user moves his mouse fast and leaves the widget deletion
dialog inadvertent, it disappears within longDuration (by default:
120) milliseconds, which is much to short to move the mouse back
and prevent disappearing.


Diffs
-

  containments/panel/contents/ui/ConfigOverlay.qml 
170aa389f8b4695ee97151081d2f2799754d0619 

Diff: https://git.reviewboard.kde.org/r/124433/diff/


Testing
---

Deleting applets or hovering over the dialog feels better now


Thanks,

David Kahles

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124481: [Panel] Fix min/max size sliderhandles

2015-07-27 Thread David Kahles

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124481/
---

(Updated July 27, 2015, 12:25 p.m.)


Status
--

This change has been marked as submitted.


Review request for Plasma.


Repository: plasma-desktop


Description
---

The x (or y) position to value calculation differed from the
value to x (or y) calulation.
This lead to strange behavior if the sliders were moved slowly (only
when panel alignment is center): Though the mouse was moved slowly,
the sliders moved very fast, sometimes even in the opposide direction.

Can I use this as a commit message or is it too verbose?


Diffs
-

  desktoppackage/contents/configuration/panelconfiguration/SliderHandle.qml 
5e8f0c42c076349e7889eeac2d1de69de40b15d5 

Diff: https://git.reviewboard.kde.org/r/124481/diff/


Testing
---

Changing the panel size (min- and maximum size) works well in all alignments.


Thanks,

David Kahles

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124485: Add missing traffic monitor

2015-07-27 Thread David Edmundson


> On July 27, 2015, 10:13 a.m., David Edmundson wrote:
> > screenshot and add usability?
> 
> Jan Grulich wrote:
> Added screenshot and usability group. I also already talked to Thomas and 
> Jens and they are fine with this change.

ah, that's all I needed to hear :)


- David


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124485/#review83026
---


On July 27, 2015, 10:17 a.m., Jan Grulich wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124485/
> ---
> 
> (Updated July 27, 2015, 10:17 a.m.)
> 
> 
> Review request for Network Management, Plasma and KDE Usability.
> 
> 
> Bugs: 268022
> http://bugs.kde.org/show_bug.cgi?id=268022
> 
> 
> Repository: plasma-nm
> 
> 
> Description
> ---
> 
> See summary.
> 
> 
> Diffs
> -
> 
>   applet/contents/ui/ConnectionItem.qml ff722a7 
> 
> Diff: https://git.reviewboard.kde.org/r/124485/diff/
> 
> 
> Testing
> ---
> 
> ![Traffic monitor](https://jgrulich.fedorapeople.org/plotter.png)
> 
> 
> Thanks,
> 
> Jan Grulich
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124485: Add missing traffic monitor

2015-07-27 Thread Jan Grulich

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124485/
---

(Updated Čec. 27, 2015, 10:17 dop.)


Review request for Network Management, Plasma and KDE Usability.


Bugs: 268022
http://bugs.kde.org/show_bug.cgi?id=268022


Repository: plasma-nm


Description
---

See summary.


Diffs
-

  applet/contents/ui/ConnectionItem.qml ff722a7 

Diff: https://git.reviewboard.kde.org/r/124485/diff/


Testing (updated)
---

![Traffic monitor](https://jgrulich.fedorapeople.org/plotter.png)


Thanks,

Jan Grulich

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124485: Add missing traffic monitor

2015-07-27 Thread Jan Grulich


> On Čec. 27, 2015, 10:13 dop., David Edmundson wrote:
> > screenshot and add usability?

Added screenshot and usability group. I also already talked to Thomas and Jens 
and they are fine with this change.


- Jan


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124485/#review83026
---


On Čec. 27, 2015, 10:06 dop., Jan Grulich wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124485/
> ---
> 
> (Updated Čec. 27, 2015, 10:06 dop.)
> 
> 
> Review request for Network Management and Plasma.
> 
> 
> Bugs: 268022
> http://bugs.kde.org/show_bug.cgi?id=268022
> 
> 
> Repository: plasma-nm
> 
> 
> Description
> ---
> 
> See summary.
> 
> 
> Diffs
> -
> 
>   applet/contents/ui/ConnectionItem.qml ff722a7 
> 
> Diff: https://git.reviewboard.kde.org/r/124485/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Jan Grulich
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124485: Add missing traffic monitor

2015-07-27 Thread David Edmundson

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124485/#review83026
---


screenshot and add usability?

- David Edmundson


On July 27, 2015, 10:06 a.m., Jan Grulich wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124485/
> ---
> 
> (Updated July 27, 2015, 10:06 a.m.)
> 
> 
> Review request for Network Management and Plasma.
> 
> 
> Bugs: 268022
> http://bugs.kde.org/show_bug.cgi?id=268022
> 
> 
> Repository: plasma-nm
> 
> 
> Description
> ---
> 
> See summary.
> 
> 
> Diffs
> -
> 
>   applet/contents/ui/ConnectionItem.qml ff722a7 
> 
> Diff: https://git.reviewboard.kde.org/r/124485/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Jan Grulich
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124453: [digital-clock] Provide a simple option for 24h clock

2015-07-27 Thread Martin Klapetek


> On July 26, 2015, 2:35 a.m., Thomas Pfeiffer wrote:
> > I fully agree with you, in both that this should only be a temporary 
> > workaround, and on that it is very helpful as such.
> > +1 from me!

What's your take on Kai's comment about using a combobox for the options? 
Currently it is a single checkbox, see the screenshot at [1]. I think you 
either want to use 24h or you want to use 12h clock and I think that one does 
not change locales that offten to warrant a "system default", a combo imho is 
then a needless clutter to the settings dialog. Comments?

[1] 
https://lh5.googleusercontent.com/-riysQFXKRfs/VbJqhhB8yBI/T4U/Zy205nGVXF0/w904-h574-no/digital-clock-5_4.png


- Martin


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124453/#review82967
---


On July 25, 2015, 11:55 p.m., Martin Klapetek wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124453/
> ---
> 
> (Updated July 25, 2015, 11:55 p.m.)
> 
> 
> Review request for Plasma and Thomas Pfeiffer.
> 
> 
> Bugs: 345378
> https://bugs.kde.org/show_bug.cgi?id=345378
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> ---
> 
> This is _the_ most requested feature for the clock, simple checkbox for 24h 
> clock.
> 
> In my opinion this is just a temporary workaround and not a very good one 
> (generally,
> code-wise it's awesome :P). There should really be a global config option for 
> that
> but as we now rely on QLocale and QLocale is...bad at this, here's at least 
> this
> small help, which will at least change the clock format on the panel.
> 
> Imo we could really really use something like KLocale again.
> 
> 
> Diffs
> -
> 
>   applets/digital-clock/package/contents/config/main.xml 8b0872b 
>   applets/digital-clock/package/contents/ui/DigitalClock.qml 9506078 
>   applets/digital-clock/package/contents/ui/configAppearance.qml ead0ad5 
> 
> Diff: https://git.reviewboard.kde.org/r/124453/diff/
> 
> 
> Testing
> ---
> 
> Switching the checkbox on and off works as expected.
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Review Request 124485: Add missing traffic monitor

2015-07-27 Thread Jan Grulich

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124485/
---

Review request for Network Management and Plasma.


Bugs: 268022
http://bugs.kde.org/show_bug.cgi?id=268022


Repository: plasma-nm


Description
---

See summary.


Diffs
-

  applet/contents/ui/ConnectionItem.qml ff722a7 

Diff: https://git.reviewboard.kde.org/r/124485/diff/


Testing
---


Thanks,

Jan Grulich

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Plasma mobile and Fairphone

2015-07-27 Thread Mario Fux
Am Sonntag, 26. Juli 2015, 22.08:04 schrieb Sebastian Kügler:

Morning Sebas

> On Sunday, July 26, 2015 21:19:13 Mario Fux wrote:
> > Am Sonntag, 26. Juli 2015, 15.18:55 schrieb Marco Martin:
> > 
> > As a Fairphone user from the first hour I'm of course all open ears here
> > (oh  and I use KDE software occasionally too ;-).
> > 
> > > Most important, if anyone has any contact to anybody working there...
> > > brilliant and let's talk
> > 
> > We were in contact with Fairphone last year about participating in the
> > Randa  Meetings which in the end didn't happen because of too many events
> > on their side. I can try and get in contact this year again. Or I can
> > introduce somebody here?
> 
> me! I'm living in the Netherlands and was already planning to contact them
> and visit their office for a coffee.

Ok, will write the introduction email this afternoon and of course CC you.

griits
Mario
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124433: [Panel] Delay disappearance of widget deletion dialog

2015-07-27 Thread David Kahles


> On July 27, 2015, 11:16 a.m., Marco Martin wrote:
> > let's make it slightly shorter, like longDuration * 3

Ok I'm fine with this.


- David


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124433/#review83020
---


On July 27, 2015, 11:53 a.m., David Kahles wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124433/
> ---
> 
> (Updated July 27, 2015, 11:53 a.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> ---
> 
> Increase the delay to 360ms, and hide the dialog immediately if
> a button on the dialog is clicked by the user.
> If a user moves his mouse fast and leaves the widget deletion
> dialog inadvertent, it disappears within longDuration (by default:
> 120) milliseconds, which is much to short to move the mouse back
> and prevent disappearing.
> 
> 
> Diffs
> -
> 
>   containments/panel/contents/ui/ConfigOverlay.qml 
> 170aa389f8b4695ee97151081d2f2799754d0619 
> 
> Diff: https://git.reviewboard.kde.org/r/124433/diff/
> 
> 
> Testing
> ---
> 
> Deleting applets or hovering over the dialog feels better now
> 
> 
> Thanks,
> 
> David Kahles
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 122488: Improved calendar navigation

2015-07-27 Thread John Layt

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122488/#review83024
---


Fantastic to see this :-) Pretty much as I documented it at 
https://community.kde.org/Plasma/Clock#Zooming_Calendar (which was a serious 
crib from Windows anyway :-) ). The thing to add for the future will be 
clicking on the day takes you to an Agenda view with your calendar and holidays 
and even weather forecast for that day :-)

- John Layt


On July 27, 2015, 10:43 a.m., Kai Uwe Broulik wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122488/
> ---
> 
> (Updated July 27, 2015, 10:43 a.m.)
> 
> 
> Review request for Plasma and KDE Usability.
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> ---
> 
> This improves the calendar navigation by providing a "Year overview" showing 
> all 12 months in a grid, and a "Decade overview" showing the current decade 
> in a grid.
> 
> A lot of code has just been moved around. The overviews use a QML ListModel 
> owing to laziness.
> 
> See https://www.youtube.com/watch?v=7SaBhRa32ds for a screencast (I love that 
> mouse click effect!)
> 
> 
> Diffs
> -
> 
>   src/declarativeimports/calendar/qml/MonthView.qml 601755f 
>   src/declarativeimports/calendar/qml/DaysCalendar.qml ab3e750 
>   src/declarativeimports/calendar/qml/DayDelegate.qml 6a3747e 
>   src/declarativeimports/calendar/daysmodel.cpp 1a6f454 
>   src/declarativeimports/calendar/daysmodel.h e1285f6 
>   src/declarativeimports/calendar/daydata.h 39ac086 
>   src/declarativeimports/calendar/calendar.h 5dc3081 
>   src/declarativeimports/calendar/calendar.cpp c462dbd 
> 
> Diff: https://git.reviewboard.kde.org/r/122488/diff/
> 
> 
> Testing
> ---
> 
> I changed the selection to be persistent during navigation; other than that, 
> should work as before, with the new overviews.
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Plasma mobile and Fairphone

2015-07-27 Thread Martin Steigerwald
Am Montag, 27. Juli 2015, 08:04:53 schrieb Sebastian Kügler:
> On Sunday, July 26, 2015 23:17:42 Martin Steigerwald wrote:
> > > Most important, if anyone has any contact to anybody working there...
> > > brilliant and let's talk
> > 
> > Well, I just contacted Fairphone support today with a forwarded mail from
> > here. I am not sure whether it makes it through to someone who can decide
> > about things, but well, its worth a try I´d say.
> > 
> > Alex and Omar are on Cc. I bet it will take at least till Monday, maybe
> > longer  as in my last experiences Fairphone support was quite loaded. I
> > own
> > a Fairphone one and I will bring it to Randa :).
> > 
> > If anyone has a more direct contact to Fairphone people that would be good
> > of  course. And if anyone from Plasma mobile team wants to be in Cc on any
> > further conversation with them, if I get a reply, then…  drop me a note.
> > 
> > (I mentioned that I am not affiliated with Plasma mobile developer team
> > and
> > am  willing to hand things over anytime.)
> 
> Can we do this in a less chaotic way, please? Mario said he has contacts, I
> said that I'll talk to the guys after being introduced by Mario, that should
> open up a channel. It's not overly useful for them to receive disjoint
> communication requests from the-same-but-not-quite directions.
> 
> Let's coordinate before acting, please.

Sebastian, I acted before I saw the mails from Mario and you. As from Alex 
response "we lack the people who could approach them" I thought there may not 
be anyone else doing it, so I just acted.

I am sorry in case you think this was not in your interest and I happily let 
it to Mario and you to contact Fairphone. I have holidays anyway, so I even 
appreciate if someone else moves it forward.

Thanks,
-- 
Martin
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124433: [Panel] Delay disappearance of widget deletion dialog

2015-07-27 Thread David Kahles

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124433/
---

(Updated July 27, 2015, 11:53 a.m.)


Review request for Plasma.


Repository: plasma-desktop


Description (updated)
---

Increase the delay to 360ms, and hide the dialog immediately if
a button on the dialog is clicked by the user.
If a user moves his mouse fast and leaves the widget deletion
dialog inadvertent, it disappears within longDuration (by default:
120) milliseconds, which is much to short to move the mouse back
and prevent disappearing.


Diffs (updated)
-

  containments/panel/contents/ui/ConfigOverlay.qml 
170aa389f8b4695ee97151081d2f2799754d0619 

Diff: https://git.reviewboard.kde.org/r/124433/diff/


Testing
---

Deleting applets or hovering over the dialog feels better now


Thanks,

David Kahles

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 122488: Improved calendar navigation

2015-07-27 Thread Kai Uwe Broulik

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122488/
---

(Updated July 27, 2015, 9:43 a.m.)


Status
--

This change has been marked as submitted.


Review request for Plasma and KDE Usability.


Changes
---

Submitted with commit 11e5ff10ae993dd2078c6019623e60d7e12f8ff1 by Kai Uwe 
Broulik to branch master.


Repository: plasma-framework


Description
---

This improves the calendar navigation by providing a "Year overview" showing 
all 12 months in a grid, and a "Decade overview" showing the current decade in 
a grid.

A lot of code has just been moved around. The overviews use a QML ListModel 
owing to laziness.

See https://www.youtube.com/watch?v=7SaBhRa32ds for a screencast (I love that 
mouse click effect!)


Diffs
-

  src/declarativeimports/calendar/qml/MonthView.qml 601755f 
  src/declarativeimports/calendar/qml/DaysCalendar.qml ab3e750 
  src/declarativeimports/calendar/qml/DayDelegate.qml 6a3747e 
  src/declarativeimports/calendar/daysmodel.cpp 1a6f454 
  src/declarativeimports/calendar/daysmodel.h e1285f6 
  src/declarativeimports/calendar/daydata.h 39ac086 
  src/declarativeimports/calendar/calendar.h 5dc3081 
  src/declarativeimports/calendar/calendar.cpp c462dbd 

Diff: https://git.reviewboard.kde.org/r/122488/diff/


Testing
---

I changed the selection to be persistent during navigation; other than that, 
should work as before, with the new overviews.


Thanks,

Kai Uwe Broulik

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124472: WIP: Implement drag and drop support for icon widget

2015-07-27 Thread Marco Martin


> On July 26, 2015, 10:46 a.m., Bhushan Shah wrote:
> > applets/icon/plugin/icon_p.cpp, line 106
> > 
> >
> > IMO it would make sense to open dropped urls only if it is supported by 
> > application..
> > 
> > like only accept text files on kate and not image files.

checking the mime supported byt the app makes sense.
it would need more C++ bindings to look at the desktop file of the app


- Marco


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124472/#review82986
---


On July 26, 2015, 10:40 a.m., Kai Uwe Broulik wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124472/
> ---
> 
> (Updated July 26, 2015, 10:40 a.m.)
> 
> 
> Review request for Plasma and Bhushan Shah.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> ---
> 
> Brings back the ability to drop files onto a icon widget and have the 
> associated application open it. Dropping something onto a folder does not yet 
> work because it should use the konq menu thingie (Copy/Move/Link) but I don't 
> want yet another copy of that (there's one in the folderview containment).
> 
> It should probably check already on drag enter whether it's a viable drop 
> target (eg. executable or folder), also do some coding style cleanups I 
> thought I had already committed earlier.
> 
> 
> Diffs
> -
> 
>   applets/icon/package/contents/ui/main.qml e0e1df7 
>   applets/icon/plugin/icon_p.h ba2d9f7 
>   applets/icon/plugin/icon_p.cpp 18806fc 
> 
> Diff: https://git.reviewboard.kde.org/r/124472/diff/
> 
> 
> Testing
> ---
> 
> Dragged Gwenview onto the desktop, an icon was created.
> Dropped one picture onto it, Gwenview opened it.
> Dropped two pictures onto, Gwenview showed both.
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Plasma 5.4 Changelog

2015-07-27 Thread Marco Martin
only thing I did on this cycle that direcctly affects desktop users is
the single engine/memory saving thing

On Mon, Jul 27, 2015 at 10:23 AM, David Edmundson
 wrote:
> Plamsa freeze is Wednesday with the release (with dot article) on the 11th
> August
>
> Can people who made notable new features in Plasma please write a short
> summary of each thing they want in the release anouncement in this thread,
> so we know what to highlight/screenshot in the article.
>
> Thanks
>
> David
>
> ___
> Plasma-devel mailing list
> Plasma-devel@kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel
>
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124481: [Panel] Fix min/max size sliderhandles

2015-07-27 Thread Marco Martin

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124481/#review83022
---

Ship it!


yes, the more detailed the better for commit messages

- Marco Martin


On July 26, 2015, 10:14 p.m., David Kahles wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124481/
> ---
> 
> (Updated July 26, 2015, 10:14 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> ---
> 
> The x (or y) position to value calculation differed from the
> value to x (or y) calulation.
> This lead to strange behavior if the sliders were moved slowly (only
> when panel alignment is center): Though the mouse was moved slowly,
> the sliders moved very fast, sometimes even in the opposide direction.
> 
> Can I use this as a commit message or is it too verbose?
> 
> 
> Diffs
> -
> 
>   desktoppackage/contents/configuration/panelconfiguration/SliderHandle.qml 
> 5e8f0c42c076349e7889eeac2d1de69de40b15d5 
> 
> Diff: https://git.reviewboard.kde.org/r/124481/diff/
> 
> 
> Testing
> ---
> 
> Changing the panel size (min- and maximum size) works well in all alignments.
> 
> 
> Thanks,
> 
> David Kahles
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124433: [Panel] Delay disappearance of widget deletion dialog

2015-07-27 Thread Marco Martin

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124433/#review83020
---

Ship it!


let's make it slightly shorter, like longDuration * 3

- Marco Martin


On July 23, 2015, 3:18 p.m., David Kahles wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124433/
> ---
> 
> (Updated July 23, 2015, 3:18 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> ---
> 
> Increase the delay to 600ms, and hide dialog immediately when
> a widget is removed by the user.
> If a user moves his mouse fast and leaves the widget deletion
> dialog inadvertent, it disappears within longDuration (by default:
> 120) milliseconds, which is much to short to move the mouse back
> and prevent disappearing.
> 
> 
> Diffs
> -
> 
>   containments/panel/contents/ui/ConfigOverlay.qml 
> 170aa389f8b4695ee97151081d2f2799754d0619 
> 
> Diff: https://git.reviewboard.kde.org/r/124433/diff/
> 
> 
> Testing
> ---
> 
> Deleting applets or hovering over the dialog feels better now
> 
> 
> Thanks,
> 
> David Kahles
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124433: [Panel] Delay disappearance of widget deletion dialog

2015-07-27 Thread David Kahles


> On July 26, 2015, 3:39 p.m., Marco Martin wrote:
> > Same concerns as the panel controller one
> 
> David Kahles wrote:
> No, this is unrelated to the panel controller. The widget deletion dialog 
> disappears even with the click to focus strategy.
> This RR only increases the delay which is already there, because this 
> makes the user experience better IHMO.
> 
> Marco Martin wrote:
> longduration*5 is really long, not sure it's a good experiance with click 
> to focus policy

There is no difference between the different focus strategies, it doesn't 
matter whether you're using click to focus or something different.
For me it feels good with longDuration*5, but I'm ok with a compromise, e.g. 
3*longDuration.
IHMO the current timeout is too short, I often move my mouse fast and then I'm 
bugged because this dialog hides so fast when I miss it with my mouse.


- David


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124433/#review82994
---


On July 23, 2015, 5:18 p.m., David Kahles wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124433/
> ---
> 
> (Updated July 23, 2015, 5:18 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> ---
> 
> Increase the delay to 600ms, and hide dialog immediately when
> a widget is removed by the user.
> If a user moves his mouse fast and leaves the widget deletion
> dialog inadvertent, it disappears within longDuration (by default:
> 120) milliseconds, which is much to short to move the mouse back
> and prevent disappearing.
> 
> 
> Diffs
> -
> 
>   containments/panel/contents/ui/ConfigOverlay.qml 
> 170aa389f8b4695ee97151081d2f2799754d0619 
> 
> Diff: https://git.reviewboard.kde.org/r/124433/diff/
> 
> 
> Testing
> ---
> 
> Deleting applets or hovering over the dialog feels better now
> 
> 
> Thanks,
> 
> David Kahles
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Plasma 5.4 Changelog

2015-07-27 Thread David Edmundson
Plamsa freeze is Wednesday with the release (with dot article) on the 11th
August

Can people who made notable new features in Plasma please write a short
summary of each thing they want in the release anouncement in this thread,
so we know what to highlight/screenshot in the article.

Thanks

David
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124458: Proofread + update khelpcenter docbook to kf5

2015-07-27 Thread Sebastian Kügler

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124458/#review83018
---

Ship it!


Ship It!

- Sebastian Kügler


On July 24, 2015, 9:20 p.m., Burkhard Lück wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124458/
> ---
> 
> (Updated July 24, 2015, 9:20 p.m.)
> 
> 
> Review request for Documentation and Plasma.
> 
> 
> Repository: khelpcenter
> 
> 
> Description
> ---
> 
> see Summary
> 
> 
> Diffs
> -
> 
>   doc/khelpcenter/index.docbook 630a366 
> 
> Diff: https://git.reviewboard.kde.org/r/124458/diff/
> 
> 
> Testing
> ---
> 
> builds
> 
> 
> Thanks,
> 
> Burkhard Lück
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124433: [Panel] Delay disappearance of widget deletion dialog

2015-07-27 Thread Marco Martin


> On July 26, 2015, 1:39 p.m., Marco Martin wrote:
> > Same concerns as the panel controller one
> 
> David Kahles wrote:
> No, this is unrelated to the panel controller. The widget deletion dialog 
> disappears even with the click to focus strategy.
> This RR only increases the delay which is already there, because this 
> makes the user experience better IHMO.

longduration*5 is really long, not sure it's a good experiance with click to 
focus policy


- Marco


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124433/#review82994
---


On July 23, 2015, 3:18 p.m., David Kahles wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124433/
> ---
> 
> (Updated July 23, 2015, 3:18 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> ---
> 
> Increase the delay to 600ms, and hide dialog immediately when
> a widget is removed by the user.
> If a user moves his mouse fast and leaves the widget deletion
> dialog inadvertent, it disappears within longDuration (by default:
> 120) milliseconds, which is much to short to move the mouse back
> and prevent disappearing.
> 
> 
> Diffs
> -
> 
>   containments/panel/contents/ui/ConfigOverlay.qml 
> 170aa389f8b4695ee97151081d2f2799754d0619 
> 
> Diff: https://git.reviewboard.kde.org/r/124433/diff/
> 
> 
> Testing
> ---
> 
> Deleting applets or hovering over the dialog feels better now
> 
> 
> Thanks,
> 
> David Kahles
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Plasma mobile and Fairphone

2015-07-27 Thread Sebastian Kügler
On Sunday, July 26, 2015 23:17:42 Martin Steigerwald wrote:
> > Most important, if anyone has any contact to anybody working there...
> > brilliant and let's talk
> 
> Well, I just contacted Fairphone support today with a forwarded mail from 
> here. I am not sure whether it makes it through to someone who can decide 
> about things, but well, its worth a try I´d say.
> 
> Alex and Omar are on Cc. I bet it will take at least till Monday, maybe
> longer  as in my last experiences Fairphone support was quite loaded. I own
> a Fairphone one and I will bring it to Randa :).
> 
> If anyone has a more direct contact to Fairphone people that would be good
> of  course. And if anyone from Plasma mobile team wants to be in Cc on any
> further conversation with them, if I get a reply, then…  drop me a note.
> 
> (I mentioned that I am not affiliated with Plasma mobile developer team and
> am  willing to hand things over anytime.)

Can we do this in a less chaotic way, please? Mario said he has contacts, I 
said that I'll talk to the guys after being introduced by Mario, that should 
open up a channel. It's not overly useful for them to receive disjoint 
communication requests from the-same-but-not-quite directions.

Let's coordinate before acting, please.
-- 
sebas

Sebastian Kügler|http://vizZzion.org| http://kde.org

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel