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

--- Comment #247 from Bharadwaj Raju <bharadwaj.raju...@protonmail.com> ---
Sorry for the delay. I'm barely able to check my KDE emails due to my other
work.

(In reply to Uwe Dippel from comment #236)
> May I ask if the features have changed since your initial activities last
> year, please? My question should be permissible, since we had been informed
> by then, that
> 1. the patch stores pre-set arrangements on a per resolution base (and not
> pop up a workable desktop at a first-time switch to a new resolution; one
> that had not been used earlier)

My patch doesn't affect what happens when a new resolution is encountered. All
it does is save different position info for each resolution, and pick the
appropriate one when a previously-encountered resolution is used again. For new
resolutions the existing behavior of trying to just fit icons is preserved.

> 2. the patch works on folder views only, not on widgets

Correct.

---

(In reply to Nick Stefanov from comment #246)
> Bharadwaj Raju tested it on my machine remotely. But it was only a files
> overwrite. I hope it'll work fine if it's compiled.

and

(In reply to tomashnyk from comment #245)
> I think this merge request really should be accepted. After living with this
> patch for ten days and not having to reorder my icons even once, I must say
> it works superbly (and actually scrollbars when there are more icons thatn
> the chosen grid supports are quite usable). I really think merging this
> would mean a much improved behaviour for a lot of users (remember the 57
> users just on this bug report). @Bharadwaj Raju said  @Nick Stefanov (and
> @Uwe Dippel ) both had troubles but neither of them has compiled the patch
> as can be seen in their comments:

The files needed for the functionality of patch are just QML. So far there are
no C++ changes other than adding debugging (which is still important, but not
to the functionality). So I don't think compiling would change the behavior.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to