question!
Best regards,
Ignat Semenov
Sebastian Kügler wrote:
Hey,
A bunch of us want to get together in May to put finishing touches in
Plasma Next. This sprint is important because it allows us to sit down
and
fix remaining bugs.
I am proposing dates here, though this is meant as a soft
on everything, as
well as general support. :)
Best regards,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
Done!
Best regards,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
?
That said, when I factor out the actions code, this is something to
consider.
Thank you!
Best regards,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
OK forget that, just talked to aseigo and he told me there still is a
usecase (and probably a few of those) for manual arrangement in the
standalone mode.
___
Plasma-devel mailing list
Plasma-devel@kde.org
way more readable, in my
opinion, not to say the it is now easier to configure the layout (I have had
troubles wrapping my head around the top to bottom right to left stuff as
well).
Best regards,
Ignat Semenov
___
Plasma-devel mailing list
Plasma
, including Q_DECLARE_METATYPE'd enums. Debugged together with
tsdgeos and steveire. So we have to workaround that, in a manually written
loop, which I put in a helper func.
Thank you for your attention!
Best regards,
Ignat Semenov
___
Plasma-devel mailing list
::fromValueTest::Foo(Test::two));
[Thursday 11 April 2013] [00:15:06] tsdgeos returns true
Best regards,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
the others, which you should have noticed, are enums with
qvariant's since that gives you compiler warnings and errors in case you
mess up - a nice thing imho :)
Thank you for the advice!
Best regards,
Ignat Semenov
___
Plasma-devel mailing list
A UI draft:
http://imagebin.org/254401
Maybe move Icon size to the Arrangement section?
Best regards,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
Bump? Any considerations / opinions / ideas?
Thank you!
Best regards,
Ignat Semenov
On Fri, Mar 29, 2013 at 11:55 PM, Ignat Semenov ragnarok...@gmail.comwrote:
Hello guys!
I've been polishing the context menu and Display page lately, both the
looks
and the code, and discovered
option in the sorting context menu?
Thank you!
Best regards,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
Hello guys!
I'd like an admin to delete the branches:
remotes/origin/fvport
remotes/origin/plasma/isemenov/folderview-qml
The first one is a mistake, the second one has stuck and we decided to
start a new one instead, ditching the old one.
Thank you!
Best regards,
Ignat Semenov
not dragging? Seems to
be completely random.
Basically, we need to change the iteration loop to something more reliable, and
that will fix the bug.
Best regards,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman
.. although the patch submitted in thei review request should go in
regardless of such a checkbox being added because at some icon sizes, the
action icons are a plain annoyance, indeed.
Thanks,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
Diff: http://git.reviewboard.kde.org/r/104396/diff/
Testing
---
Tested, works.
Maybe we should also stop automatic scrolling if the user releases the LMB
while scrollling automatically as a result of dragging the rubberband over the
view border? Fredrik, what do you think?
Thanks,
Ignat
scrollling automatically as a result of dragging the rubberband over the
view border? Fredrik, what do you think?
Thanks,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
.
Diffs (updated)
-
plasma/applets/folderview/iconview.h 12e93b3
plasma/applets/folderview/iconview.cpp 5c4e086
Diff: http://git.reviewboard.kde.org/r/103822/diff/
Testing
---
Testing done against master, seems to behave indentically.
Thanks,
Ignat Semenov
(updated)
-
plasma/applets/folderview/folderview.h 4e441eb
plasma/applets/folderview/folderview.cpp 121d1e7
Diff: http://git.reviewboard.kde.org/r/104328/diff/
Testing
---
Tested, works.
Thanks,
Ignat Semenov
___
Plasma-devel mailing list
(updated)
-
plasma/desktop/applets/kickoff/ui/contextmenufactory.cpp 99c2649
Diff: http://git.reviewboard.kde.org/r/103830/diff/
Testing
---
Tested, works.
Thanks,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https
in the desktop containment and performs a KIO::link() if that's the case.
Diffs (updated)
-
plasma/desktop/applets/kickoff/ui/contextmenufactory.cpp 8db9655
Diff: http://git.reviewboard.kde.org/r/103830/diff/
Testing
---
Tested, works.
Thanks,
Ignat Semenov
121d1e7
Diff: http://git.reviewboard.kde.org/r/104328/diff/
Testing
---
Tested, works.
Thanks,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
, this should also abort restoring the position.
Also, I'm wondering if we really need to save the position separately for
the iconview and the listview, or if we should use the same key.
Otherwise the patch looks good to me.
Ignat Semenov wrote:
Well I've been thinking about
---
Tested, works.
Would be nice to give the code a static analyzer run, if there was a decent
static analyzer available in Linux. Anybody have relevant experience?
Thanks,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https
://git.reviewboard.kde.org/r/104311/diff/
Testing
---
Tested, works.
Would be nice to give the code a static analyzer run, if there was a decent
static analyzer available in Linux. Anybody have relevant experience?
Thanks,
Ignat Semenov
___
Plasma-devel mailing
view and the list view, works fine.
Thanks,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
/folderview/listview.cpp 94efe44
Diff: http://git.reviewboard.kde.org/r/104258/diff/
Testing
---
Tested both the icon view and the list view, works fine.
Thanks,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org
/
Testing
---
Tested both the icon view and the list view, works fine.
Thanks,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
, this should also abort restoring the position.
Also, I'm wondering if we really need to save the position separately for
the iconview and the listview, or if we should use the same key.
Otherwise the patch looks good to me.
Ignat Semenov wrote:
Well I've been thinking about
authentication just to browse the
existing requests read-only. Is it possible to do it differently or is this
a shortcoming of gerrit ( or its design feature)? Pretty user- and
newcomer-unfriendly, too.
Best regards,
Ignat Semenov
___
Plasma-devel mailing list
Ben Cooksley wrote:
On Tue, Mar 13, 2012 at 7:58 PM, Ignat Semenov ragnarok...@gmail.com
wrote:
Hello fellow KDE devs!
While I'm not an experienced developer nor manager, the planned
transition to gerrit really troubles me. In particular, I have the
following questions:
1)The gerrit
plasma/applets/folderview/listview.cpp b17e7c4
Diff: http://git.reviewboard.kde.org/r/104258/diff/
Testing
---
Tested both the icon view and the list view, works fine.
Thanks,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https
and the list view, works fine.
Thanks,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
plasma/applets/folderview/folderview.cpp a94ce87
plasma/applets/folderview/iconview.cpp 5c4e086
plasma/applets/folderview/listview.cpp b17e7c4
Diff: http://git.reviewboard.kde.org/r/104258/diff/
Testing
---
Tested both the icon view and the list view, works fine.
Thanks,
Ignat Semenov
to implement.
- Ignat
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104258/#review11354
---
On March 13, 2012, 1:51 p.m., Ignat Semenov
, this should also abort restoring the position.
Also, I'm wondering if we really need to save the position separately for
the iconview and the listview, or if we should use the same key.
Otherwise the patch looks good to me.
Ignat Semenov wrote:
Well I've been thinking about
b17e7c4
Diff: http://git.reviewboard.kde.org/r/104258/diff/
Testing
---
Tested both the icon view and the list view, works fine.
Thanks,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo
?
- Ignat
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103830/#review10265
---
On Jan. 31, 2012, 3:45 p.m., Ignat Semenov wrote
in the meanwhile.
So I'd like to ask you guys, especially Aaron:have you fixed any
containment sizing related bugs recently? Anything in the desktop
containment or deeper in libplasma?
If not, I'll just dig into the FW code to try and figure stuff out..
Best regards,
Ignat Semenov
my work on the automatic detection.
Opinions? This may look like a minor issue not worth sucha long email and
such a long discussion, but these minor nitpicks, or lack thereof, do sum up
to crate a positive user experience.
Best regards
Ignat Semenov
be done.
Waiting for two people (mostly one) to come up on IRC to finish your code is
not normal ither.
Thus, I encourage those in the know to document this area with examples and
a real explanation, as seen in the Qt docs. Thank you a lot in advance.
Best regards,
Ignat Semenov
impossible to put applets in the
necessary column / row, ipossible to control applets' size, applets
overlapping each other, etc. I can make a screencast if you wish, but you
most probably can reproduce it yourself.
Best regards,
Ignat Semenov
___
Plasma
', it was sometimes impossible to put applets in the
necessary column / row, ipossible to control applets' size, applets
overlapping each other, etc. I can make a screencast if you wish, but you
most probably can reproduce it yourself.
Best regards,
Ignat Semenov
can reproduce it yourself.
Best regards,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
::themeChanged(), an unconditional one.)
So, I'd like to avoid this second repaint (via update()), and reload
the framesvg manually. How can I do that?
Best regards,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman
,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
---
Tested, builds and works as described above.
Thanks,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
Marco Martin wrote:
On Mon, Feb 6, 2012 at 9:22 AM, Ignat Semenov ragnarok...@gmail.com
wrote:
Hello fellow plasma devs!
This change, while trivial, contains 2 strings, Ascending and
Desending. I've found out that the very same strings already exist in
Dolphin, which lives in the same
/103860/diff/diff
Testing
---
Tested, works. Changes the icon sort direction, even though the actual sorting
is sometimes broken (my next task, most probably.)
Thanks,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https
/folderview.h e458d77
plasma/applets/folderview/folderview.cpp 54ea14a
Diff: http://git.reviewboard.kde.org/r/103860/diff/diff
Testing
---
Tested, works. Changes the icon sort direction, even though the actual sorting
is sometimes broken (my next task, most probably.)
Thanks,
Ignat Semenov
: http://git.reviewboard.kde.org/r/103860/diff/diff
Testing
---
Tested, works. Changes the icon sort direction, even though the actual sorting
is sometimes broken (my next task, most probably.)
Thanks,
Ignat Semenov
___
Plasma-devel mailing list
0243b2f
plasma/applets/folderview/popupview.cpp 7fdb3f7
Diff: http://git.reviewboard.kde.org/r/103851/diff/diff
Testing
---
Tested, works.
Thanks,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman
is used
in the desktop containment and performs a KIO::link() if that's the case.
Diffs
-
plasma/desktop/applets/kickoff/ui/contextmenufactory.cpp 8db9655
Diff: http://git.reviewboard.kde.org/r/103830/diff/diff
Testing
---
Tested, works.
Thanks,
Ignat Semenov
()
signal.
Diffs
-
plasma/widgets/iconwidget.cpp 1161cc4
Diff: http://git.reviewboard.kde.org/r/103831/diff/diff
Testing
---
Tested on desktop and in the panel with KDE set to both double- and
single-click, works fine.
Thanks,
Ignat Semenov
,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
://git.reviewboard.kde.org/r/103822/diff/diff
Testing
---
Testing done against master, seems to behave indentically.
Thanks,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
://git.reviewboard.kde.org/r/103735/diff/diff
Testing
---
Tested by manually changing the paintItem() code to use the itemSize() rect and
observing the improved behaviour.
Thanks,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https
the paintItem() code to use the itemSize() rect and
observing the improved behaviour.
Thanks,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
---
Tested by manually changing the paintItem() code to use the itemSize() rect and
observing the improved behaviour.
Thanks,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
:38 p.m., Ignat Semenov wrote:
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103735/
---
(Updated Jan. 19, 2012, 2:38 p.m
, works both with KDe set ot single- and double-click.
Thanks,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
/applets/folderview/iconview.h 38cad54
plasma/applets/folderview/iconview.cpp 842e158
Diff: http://git.reviewboard.kde.org/r/103679/diff/diff
Testing
---
Tested, works both with KDe set ot single- and double-click.
Thanks,
Ignat Semenov
/diff
Testing
---
This is not final, there is 1 pixel offset bug in the halo drawing code. I'm
going to sleep today, this review request is just to show that I'm aware that I
have screwed things up and am working on getting the proper patch done.
Thanks,
Ignat Semenov
, please, have a look into this.
This happens with any settings, not onyl the ones I added. (My settings code is
absolutely identical to the Click to view folder settigns code).
Thanks,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
only on Plasma restart and not on the
fly after hitting the Appy or Ok buttons. Aaron, please, have a look into this.
This happens with any settings, not onyl the ones I added. (My settings code is
absolutely identical to the Click to view folder settigns code).
Thanks,
Ignat Semenov
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103618/#review9500
---
On Jan. 3, 2012, 1:03 p.m., Ignat Semenov wrote
.
Ignat Semenov wrote:
OK. So, are you fine with the implementation code?
If yes, I will commit the backend part (ActionOverlay) separately? The
GUI bits may come in later.
Please, give me a final decision on this. Do I maybe need to wait for
Aaron to approve
).
Thanks,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
and not on the
fly after hitting the Appy or Ok buttons. Aaron, please, have a look into this.
This happens with any settings, not onyl the ones I added. (My settings code is
absolutely identical to the Click to view folder settigns code).
Thanks,
Ignat Semenov
. Peter, we'd like
to know the rationale for exposing such an option in the configuration UI.
- Ignat Semenov
On Jan. 3, 2012, 5:21 p.m., Ignat Semenov wrote:
---
This is an automatically generated e-mail. To reply, visit:
http
settings, which is
perfectly fine per se. But, now when we click Apply or OK, the
settings are written to the disk, but are **not** applied. Tested in
trunk. This needs some solution as well.
Best regards,
Ignat Semenov
___
Plasma-devel mailing list
is that the timer call is there in order to allow the
newly written settings to sync to the hard disk, so I put the call at the very
end of the accepted() method.
Thanks,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org
an hour of normal use, no crashes except for the
mentioned crash in the config dialog.
Regards,
Ignat Semenov
P.S. As you say, nouveau allows for GLES (didn't know about that, btw.) Can
test this as well if necessary. (Not sure about the prerequisites, that
could take some time
things earlier that later anyway, don't you think? And
mind you I'm from Russia, but that should not be a problem anyway, hope so
at least.
Cheers,
Ignat Semenov aka thorGT
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman
to investigate and the least amount of lines to submit after solving
the problem, don't you think.)
Cheers,
Ignat Semenov
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
On 5/25/10, Aaron Seigo ase...@kde.org wrote:
so it's not crammed to the left side as before
which is how all the other dialogs are. please don't change one dialog to
some random new style without first discussing it with us on
plasma-de...@kde.org. thanks. if we change this one, then they
---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4154/
---
Review request for Plasma and Aaron Seigo.
Summary
---
This patch makes
And maybe no text at all for the separators? Like one more font face
and not aligned with anything either.
(Yes, I like to remove stuff ;)
On Tue, May 25, 2010 at 8:22 PM, Aaron J. Seigo ase...@kde.org wrote:
On May 22, 2010, Jacopo De Simoi wrote:
I'm playing around with that now, and here's
---
On 2010-05-23 22:52:04, Ignat Semenov wrote:
---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4119/
---
(Updated 2010-05-23 22:52:04
---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4124/
---
(Updated 2010-05-25 18:07:48.304935)
Review request for kdelibs and Plasma.
---
On 2010-05-25 18:07:48, Ignat Semenov wrote:
---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4124/
---
(Updated 2010-05-25 18:07:48
, visit:
http://reviewboard.kde.org/r/4130/#review5860
---
On 2010-05-24 18:57:54, Ignat Semenov wrote:
---
This is an automatically generated e-mail. To reply, visit:
http
.
Ignat Semenov wrote:
Please, show me a config dialog with a separator in KDE. No offense, but
I really haven't found one. Again, it creates visual noise and again, we
agreed on removing it.
And I have no SVN account atm, should I apply for one?
- Ignat
---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4124/
---
(Updated 2010-05-24 16:55:36.267182)
Review request for kdelibs and Plasma.
---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4130/
---
Review request for Plasma and Marco Martin.
Summary
---
This is further
Hi all,I'm currently trying to improve the Pager Configuration Dialog. Actually, it's not the only one that should be improved IMHO, but well, that's a different story, now I'm fighting the Pager plasmoid. So far so good. I've decided to do this :- Change radiobuttons to comboboxes, as any
Nevermind, I've cheated and it works now :)Was as simple aswidget-setLayout(ui.verticalLayout);in createConfigurationInterface().So here's the patch version 1. plasma-devel says "no 4o KB messages", so second patch comes in the second letter.Please tell me what you like more and what fits
---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4119/
---
Review request for Plasma.
Summary
---
This is the final version of the
---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4119/
---
(Updated 2010-05-23 18:32:54.867447)
Review request for Plasma.
Changes
---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4119/
---
(Updated 2010-05-23 22:52:04.976831)
Review request for Plasma.
Summary
On Sat, May 22, 2010 at 1:44 PM, Jacopo De Simoi wilder...@gmail.com wrote:
Well, as far as device notifier goes, there's a couple more things about it
that I'd like to see fixed as well.
Here I am, the device notifier mantainer, at your service :)
Nice to meet you :)
This one about
---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4086/
---
Review request for Plasma.
Summary
---
After launching a query result from
. To reply, visit:
http://reviewboard.kde.org/r/4086/#review5796
---
On 2010-05-21 12:27:30, Ignat Semenov wrote:
---
This is an automatically generated e-mail. To reply, visit:
http
---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4100/
---
Review request for Plasma.
Summary
---
Currently, device notifier tooltips
On 2010-05-21 12:42:14, Marco Martin wrote:
yes, those are that little annoyance that make the difference :)
Ignat Semenov wrote:
Don't have an SVN account yet; could you commit this for me?
Ping? Is this committed?
- Ignat
---
On 2010-05-21 15:15:38, Ignat Semenov wrote:
---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4100
96 matches
Mail list logo