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

Added "Randomize Marker Color" option from #2210 #2333

Open
wants to merge 3 commits into
base: next
Choose a base branch
from

Conversation

Azagwen
Copy link

@Azagwen Azagwen commented May 12, 2024

This pull adds a "random marker color" option in the outliner context menu, which will pick a random color (different from the current one) and apply it to any selected element

image

demo video:

electron_dMGeebxFbF.mp4

Copy link

netlify bot commented May 12, 2024

Deploy Preview for blockbench-dev ready!

Name Link
🔨 Latest commit 0c2c84b
🔍 Latest deploy log https://app.netlify.com/sites/blockbench-dev/deploys/66411c6d0451590008e617e5
😎 Deploy Preview https://deploy-preview-2333--blockbench-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@Azagwen Azagwen changed the title Added suggestion from #2210 Added "Randomize Marker Color" option from #2210 May 14, 2024
@JannisX11
Copy link
Owner

A few things to improve:

  • Let's implement this as a standalone Action instead of a menu item. That way users can add it to their toolbars and assign keybindings. Actions can be added back to menus by adding their ID as a string
  • Let's change the name to "Randomize Marker Colors". That way the action is more clear, and the plural clarifies that each element gets its own color
  • Let's change the icon to "fa-shuffle"
  • I haven't fully made up my mind about it, but it might be neat to move this action directly into the Marker Color submenu, below the other points. That way the main menu is less cluttered. But lmk your thoughts.

@Azagwen
Copy link
Author

Azagwen commented Jun 2, 2024

Will do these changes when I find the time 👍

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.

None yet

2 participants