[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: RE: Desktop Slideshow Settings
From: Steven Sroka <steven.sroka () live ! ca>
Date: 2010-12-08 7:10:19
Message-ID: BLU146-W592E2E2A76045927E62E9952D0 () phx ! gbl
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
In kdebase/workspace/libs/plasmagenericshell/backgrounddialog.cpp, what happens in \
BackgroundDialog::saveConfig() specifically?
In general, I know that the wallpaper/mouse settings are saved and the last 'if' \
statement forces the wallpaper to change. But, what does the code above the last 'if' \
statement *specifically* do?
Steve
From: aseigo@kde.org
To: plasma-devel@kde.org
Subject: Re: Desktop Slideshow Settings
Date: Sun, 5 Dec 2010 23:01:33 -0800
On Sunday, December 5, 2010, Steven Sroka wrote:
> add at the very beginning of BackgroundDialog::saveConfig(),if(
> !hasChanged() ) { return;}orif( hasChanged() ) { //the existing code that
> is run when a setting is changed.}
yes, that would work. it would still mean the wallpaper would reload if the
layout type was changed (e.g. default dekstop -> folderview) or when the mouse
actions are changed.
so while what you suggest would be an improvement (and i'd probably accept
such a patch), it would be even nicer to track wallpaper changes independantly
as well.
--
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA EE75 D6B7 2EB1 A7F1 DB43
KDE core developer sponsored by Qt Development Frameworks
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
[Attachment #5 (text/html)]
<html>
<head>
<style><!--
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 10pt;
font-family:Tahoma
}
--></style>
</head>
<body class='hmmessage'>
<style><!--
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 10pt;
font-family:Tahoma
}
--></style>
In kdebase/workspace/libs/plasmagenericshell/backgrounddialog.cpp, what happens in \
BackgroundDialog::saveConfig() specifically?<br><br>In general, I know that the \
wallpaper/mouse settings are saved and the last 'if' statement forces the wallpaper \
to change. But, what does the code above the last 'if' statement *specifically* \
do?<br><br><br>Steve<br><br><hr><br>From: aseigo@kde.org<br>To: \
plasma-devel@kde.org<br>Subject: Re: Desktop Slideshow Settings<br>Date: Sun, 5 Dec \
2010 23:01:33 -0800<br><br><pre>On Sunday, December 5, 2010, Steven Sroka \
wrote:<br>> add at the very beginning of \
BackgroundDialog::saveConfig(),if(<br>> !hasChanged() ) { return;}orif( \
hasChanged() ) { //the existing code that<br>> is run when a setting is \
changed.}<br> <br>yes, that would work. it would still mean the wallpaper would \
reload if the <br>layout type was changed (e.g. default dekstop -> folderview) or \
when the mouse <br>actions are changed.<br> <br>so while what you suggest would be an \
improvement (and i'd probably accept <br>such a patch), it would be even nicer to \
track wallpaper changes independantly <br>as well.<br> <br>-- <br>Aaron J. \
Seigo<br>humru othro a kohnu se<br>GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA EE75 \
D6B7 2EB1 A7F1 DB43<br> <br>KDE core developer sponsored by Qt Development \
Frameworks<br></pre><br>_______________________________________________ Plasma-devel \
mailing list Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
</body>
</html>
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic