In D28057#657036, @ngraham wrote:Re-reviewing this is on my to-do list. Sorry for the delay!
- Queries
- All Stories
- Search
- Advanced Search
Feed Advanced Search
Advanced Search
Advanced Search
Apr 25 2020
Apr 25 2020
pereira.alex added a comment to D28057: Fix folderview popup mode list view icon size inheriting icon view icon size.
pereira.alex added a comment to D28057: Fix folderview popup mode list view icon size inheriting icon view icon size.
@ngraham did I do something wrong ? If i did it was not intentional.
Apr 19 2020
Apr 19 2020
pereira.alex updated the diff for D28057: Fix folderview popup mode list view icon size inheriting icon view icon size.
Trying to fix the bug:
Apr 10 2020
Apr 10 2020
pereira.alex added a comment to D28057: Fix folderview popup mode list view icon size inheriting icon view icon size.
In D28057#645164, @ngraham wrote:I would give each view mode its own config value and adjust the code with conditionals; that's not a problem. There are no backwards compatibility concerns here if you do that, except for people currently using list view with gigantic icons, which will become small, fixing the bug. If they complain, we can just tell them that it was a bug and if they want huge list icons again, they'll have to configure them that way deliberately. :)
Alternatively we can give up on allowing the icon size in list view to be configured and just always use a small size.
pereira.alex added a comment to D28057: Fix folderview popup mode list view icon size inheriting icon view icon size.
In D28057#645140, @ngraham wrote:I still don't think we need three values in the config file for only two view modes.
Mar 31 2020
Mar 31 2020
pereira.alex added a comment to D28057: Fix folderview popup mode list view icon size inheriting icon view icon size.
In D28057#639054, @ngraham wrote:Sorry for the delay, I will resume reviewing this soon.
Mar 28 2020
Mar 28 2020
I don't want to be *that guy* but this is one of the things I do in every kde desktop I have. User facing configurable UI would be great !
pereira.alex updated the diff for D28057: Fix folderview popup mode list view icon size inheriting icon view icon size.
Deal with upgrading configuration
pereira.alex added a comment to D28057: Fix folderview popup mode list view icon size inheriting icon view icon size.
Also forgot to mention:
pereira.alex updated the diff for D28057: Fix folderview popup mode list view icon size inheriting icon view icon size.
Use single slider and sync values
Mar 25 2020
Mar 25 2020
pereira.alex added a comment to D28057: Fix folderview popup mode list view icon size inheriting icon view icon size.
In D28057#634650, @ngraham wrote:Abandoning the existing iconSize config value and adding a new one called gridViewIconSize means that everyone'e existing icon view size will be reset, since nothing will be reading from the iconSize values in the config files anymore. You have two options here:
- Don't abandon iconSize for the icon view; keep using it even though its name isn't perfectly descriptive anymore
- Create a kconf update script to migrate people's config files so that the existing iconSize values get transformed into gridViewIconSize See https://techbase.kde.org/Development/Tools/Using_kconf_update
Mar 22 2020
Mar 22 2020
pereira.alex updated the diff for D28057: Fix folderview popup mode list view icon size inheriting icon view icon size.
Fixing bug on changing icon size between list/grid view
Mar 18 2020
Mar 18 2020
pereira.alex added a comment to D28057: Fix folderview popup mode list view icon size inheriting icon view icon size.
I did the changes above like you said in the patch above.
I didn't applied it yet, because on testing that same patch, I discovered a bug. I reverted the changes ( after saving the work in the above patch ) and tested the current implementation and it also has the bug.
Mar 17 2020
Mar 17 2020
pereira.alex updated the diff for D28057: Fix folderview popup mode list view icon size inheriting icon view icon size.
Using separate configs for list and icon mode
pereira.alex added a comment to D28057: Fix folderview popup mode list view icon size inheriting icon view icon size.
In D28057#628853, @ngraham wrote:Thanks!
Now we have a new problem though: the default icon size for list view is synchronized with the default icon size for icon view, so it's now gigantic by default:
We need for the default value itself to be different for icon vs list view, while allowing each one to be independent. Probably the cleanest way to do this is to define a new value in the config file for the default list view icon size (could be called something like listViewIconSize) and then change makeIconSize() (good thing you kept it, lol) to return return FolderTools.iconSizeFromTheme(plasmoid.configuration.listViewIconSize) when it's in list view.
Does that make sense?
pereira.alex added a comment to D11977: Change the default animationTime of 350 to 250, like most other effects.
In D11977#628832, @ngraham wrote:In D11977#257168, @mart wrote:250 is fine, and yeah, on x11 risks more to stutter.
what would be needed is perhaps having a single, kdeglobals level setting for animation duration.Oh look, we have that now. :)
@pereira.alex, maybe you would be interested in porting this effect (and perhaps others too) to read the duration from that setting?
pereira.alex updated the diff for D28057: Fix folderview popup mode list view icon size inheriting icon view icon size.
Remove visible: true since its default
Mar 16 2020
Mar 16 2020
thank you all for the positive feedback :)
pereira.alex updated the diff for D28057: Fix folderview popup mode list view icon size inheriting icon view icon size.
Removed commented code
pereira.alex added a comment to D28055: Fix on wayland activating minimized window will move window to current desktop.
In D28055#628705, @zzag wrote:it will put window to VD1 and VD2.
It seems to me like this bug must be fixed in KWin rather than the task manager.
Mar 15 2020
Mar 15 2020
pereira.alex removed a reviewer for D28055: Fix on wayland activating minimized window will move window to current desktop: VDG.
Trying to add the correct reviewers :)
More info: as discussed here: https://www.reddit.com/r/kde/comments/fgwcou/tip_animation_speedslider/
Apr 23 2018
Apr 23 2018
In D11976#251820, @graesslin wrote:The idea how the glide effect was implemented is to have this angle. The defaults have not been changed since the effect got introduced.
As this is an effect not enabled by default I do not see the need to change any default settings.
Apr 6 2018
Apr 6 2018
In D11976#240941, @abetts wrote:As you work through this, can you please also upload videos to understand the changes visually too?
In D11976#240923, @zzag wrote:In D11976#240922, @pereira.alex wrote:Tilt nowadays may be too much wobbly , and being default the user might be put off immediatly. Btw tilt was not one of the original purposes of Glide. At least not when I developed it.
So, it has nothing to do with http://wiki.compiz.org/Plugins/Animation#Glide_1 ?
Apr 5 2018
Apr 5 2018
pereira.alex added a comment to D11977: Change the default animationTime of 350 to 250, like most other effects.
In D11977#240918, @zzag wrote:Why another patch? Because it's settings related, you could change animation duration in D11976.
In D11976#240917, @zzag wrote:In D11976#240906, @pereira.alex wrote:It was to be similar to scale effect, but scale effect only "appears", doesnt animate closing. Also with the purpose to "appear from below" or "appear from above" to be configurable. I actually never used it as "rotation of windows"!
-1, sorry. (please notice that I'm not a member of KWin so you shouldn't worry what I think ;-)
Having tilt is one of the purposes(or key components) of the glide effect. IIRC Glide 1 from Compiz also has some angle.
I made two change requests:
https://phabricator.kde.org/D11976 and
https://phabricator.kde.org/D11977
In D11976#240900, @zzag wrote:glide effect to remove the angle and appear from above and disappear to below
I thought that was the point of the glide effect.
With these new params it looks pretty much like scale effect.
I have been away from development for a long time. It was very diferently in the kde3 era :)
As the initial author of the glide effect ( then code was cleaned and improved by the kwin team ), there was something I should have done at the time : Change the defaults !
I never really planned it to be like in the default manner.
pereira.alex added a comment to T7911: Kubuntu should install PowerTOP and TLP and turn them on during installation : Publicity & Testing needed; re-evaluate for 18.10.
I am new here, hope i am not out of order by replying here but using TLP before made my desktop computer freeze randomly from time to time. It was quite a hard to figure out it was TLP's fault because it was quite random ( it seems it was a problem between TLP and ssd disk ).