Skip to content

Unit improvements#96

Merged
pascalpp merged 15 commits intomasterfrom
unit-updates
Feb 3, 2021
Merged

Unit improvements#96
pascalpp merged 15 commits intomasterfrom
unit-updates

Conversation

@pascalpp
Copy link
Owner

  • adds a Cycle Units menu command with shortcut U
  • adds unit labels in the ruler for PX, MM, and IN
  • improves mouse tick drawing code

image

image

image

image

@pascalpp pascalpp requested a review from jeff-h January 29, 2021 03:44
<connections>
<action selector="toggleGroupRulers:" target="Voe-Tx-rLC" id="RkN-cz-hMe"/>
</connections>
</menuItem>
Copy link
Owner Author

Choose a reason for hiding this comment

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

this was a weird stray menuitem that somehow got added outside the menu, not visible in the app. A duplicate of the existing Group Rulers menu item.

fixes a bug where the unit label would disappear when dragging the other ruler
@pascalpp
Copy link
Owner Author

pascalpp commented Feb 3, 2021

@jeff-h Do you have some time to try out this PR before I merge it out?

@pascalpp
Copy link
Owner Author

pascalpp commented Feb 3, 2021

@carlfriess Do you have some time to try out this PR before I merge it out?

@carlfriess
Copy link
Contributor

Looks good to me! :)

Co-authored-by: Carl Friess <carl.friess@me.com>
@pascalpp pascalpp merged commit a65e86f into master Feb 3, 2021
@pascalpp pascalpp deleted the unit-updates branch February 3, 2021 17:00
@jeff-h
Copy link
Collaborator

jeff-h commented Feb 3, 2021

Hey, sorry I haven't had a chance to look at anything properly recently — just been flat out with other work. Appreciate all your work though! These latest features are awesome.

@pascalpp
Copy link
Owner Author

pascalpp commented Feb 4, 2021

@jeff-h no worries!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants