Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make custom pos checkbox work for multiple items #499

Merged
merged 1 commit into from May 10, 2017

Conversation

tsujan
Copy link
Member

@tsujan tsujan commented May 1, 2017

Fixes #498.

The state of the "Stick to current position" checkbox is corrected for multiple selections. Also when multiplely selected desktop items are right-clicked and the box is checked, all of them stick to their positions; if the box is unchecked, all of them will be sorted according to the sorting order.

Fixes #498.

The state of the "Stick to current position" checkbox is corrected for multiple selections. Also when multiplely selected desktop items are right-clicked and the box is checked, all of them stick to their positions; if the box is unchecked, all of them will be sorted according to the sorting order.
@tsujan tsujan requested review from PCMan and agaida May 1, 2017 07:44
tsujan added a commit that referenced this pull request May 7, 2017
Fixes (a part of) #500.

For a more natural "desk-top" experience, dropped items are positioned successively (top→bottom + left→right), starting with the drop position (rectangle).

IMO, there's no need to make this optional because mose users expect such a behavior and if a user doesn't want it, he/she could use the context menu easily (see #499).
tsujan added a commit that referenced this pull request May 7, 2017
Fixes (a part of) #500.

For a more natural "desk-top" experience, dropped items are positioned successively (top→bottom + left→right), starting with the drop position (rectangle).

IMO, there's no need to make this optional because most users expect such a behavior and if a user doesn't want it, he/she could use the context menu easily (see #499).
Copy link
Member

@agaida agaida left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, works as expected - but i still think we should have an option to fix all objects in the settings - it look strange when some objects are not fixed. https://youtu.be/ihvDaVvhsEo

@tsujan
Copy link
Member Author

tsujan commented May 10, 2017

@agaida
I think the purpose of this PR is misunderstood. Let me explain it:

There are some items that have custom positions. When you right click one of them, "Stick to Current Position" is checked. Other items don't have custom positions but are placed according to the sorting method. For them, the checkbox in question isn't checked, of course.

Previous to this PR, when you right-clicked multiple items and checked/unchecked the box, they wouldn't all get/lose fixed positions -- it worked only with a single item. That was definitely a bug, which is fixed by this PR.

EDIT: Corrected the last paragraph.

@tsujan
Copy link
Member Author

tsujan commented May 10, 2017

@agaida
You might ask, "why do some items have custom/fixed positions?" The answer is because the user has given custom positions to them by either (1) dragging and dropping them to some other places, or (2) right clicking on them and checking "Stick to Current Position".

@tsujan
Copy link
Member Author

tsujan commented May 10, 2017

Yet another explanation regarding your screen-cast: When you copy items to desktop, they don't have custom positions yet, so they're positioned according to the sorting order. The case of drag-n-drop was different, so I made #504 for it.

BTW, I have a fix for those ugly item jumps that can be seen in your screen-cast too. Waiting for these PRs to be merged because I hate unneeded rebasing ;)

@agaida agaida merged commit 785a354 into master May 10, 2017
@agaida agaida deleted the multilple_custom_pos branch May 10, 2017 08:09
agaida pushed a commit that referenced this pull request May 10, 2017
Fixes (a part of) #500.

For a more natural "desk-top" experience, dropped items are positioned successively (top→bottom + left→right), starting with the drop position (rectangle).

IMO, there's no need to make this optional because most users expect such a behavior and if a user doesn't want it, he/she could use the context menu easily (see #499).
@agaida agaida added this to Done in Pull Requests Mar 28, 2018
@agaida agaida added this to Old in PRs Nov 5, 2018
@agaida agaida added this to Old Archive in Pull Requests Mar 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
PRs
  
Changelog
Pull Requests
  
Old Archive (fucked up)
Development

Successfully merging this pull request may close these issues.

None yet

2 participants