bulia byak wrote:
Hi guys, overall you did a terrific job on these features,
Definitely true!
- "snap guides while dragging" option does not seem to work - no
snapping to grid, other guides or objects
Yes, it does. At first, I was thinking the same - until I read the tooltip. It explains very well how to make this option work. However, IMO the problem is that the user normally expects a different behaviour and thus doesn't bother to read the tooltip.
I personally find it somewhat unintuitive that guides don't snap automatically when "Snap guides while dragging" is checked. I wonder if it would make sense to snap guides unconditionally in this case (or at least to nodes by default), without the need to specify the precise snapping behaviour in the "Snaps" tab (we could provide a keyboard modifier to temporarily avoid snapping while dragging guides).
Otherwise, we definitely need to point the user more explicitly to the "Snaps" tab - either by adding a comment (outside the tooltip) in the "Guides" tab or by moving the said option into the "Snaps" tab, too (i.e., under the section "What snaps" add a checkbox for guidelines).
- can we disable the remove button when there's nothing to remove in Grids?
This comment I don't understand. I only see this button when a grid is present in the document, and in this case it successfully removes the grid.
- why two checkboxes, "enabled" and "visible"? what's the difference?
It's not clear. I propose to rename them to "Snapping enabled" and "Grid visible". Also if I uncheck "enabled", visible does not work - why? if that's intentional, please just gray out the rest of the grid options when "enabled" is unchecked.
Another related issue: Is the '#' shortcut supposed to only toggle visibility or to also enable/disable grids? Currently it does both (which is what I would expect, so I'd leave it that way), but
1) it doesn't update the corresponding checkboxes in the "Grids" tab 2) this behaviour seems to contradict the fact that the shortcut is listed in the "View" menu (which suggests that it only toggles visibility)
Max