Skip to content

Conversation

@choosingausernameishard
Copy link
Contributor

Here’s how they look in context:

Screenshot 2025-08-15 at 09 16 51

Screenshot 2025-08-15 at 09 17 42

Copy link
Member

@danirabbit danirabbit left a comment

Choose a reason for hiding this comment

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

Thanks for submitting your branch! Can you please submit two separate branches for the lock/unlock symlinks and the hidden/visible icons? It would help keep the commit history cleaner and be easier to review to not lump together different metaphors

We have an unlocked icon already in status called changes-allow. I'm happy with adding more symlinks to the existing icon, but I don't want to have two separate unlocked icons. If you want to propose changes to the "unlocked" icon, I think I'd also like to see separate branches for proposing design changes vs adding symlinks

@choosingausernameishard
Copy link
Contributor Author

choosingausernameishard commented Aug 16, 2025

I have now split this PR into three different pull requests, as requested:

I had also submitted another Inkscape-related pull request. Let me know if it should be split as well:

@choosingausernameishard choosingausernameishard deleted the inkscape-eye-and-lock branch September 1, 2025 10:54
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.

2 participants