Skip to content

Conversation

@BeckerWdf
Copy link
Member

Fixes: #137

@BeckerWdf
Copy link
Member Author

@jasmin261098 can you pls. review?

@jasmin261098
Copy link
Contributor

Thank you for the contribution.
The following two adjustments are still necessary to comply with the style guide:

  • The spacing of the cut-out lines should be 1.5 px; at the moment it is 1 px.
Image
  • The top right corner is correctly cut, but the corners should also be rounded, as shown in the screenshot:
Image

The process in Inkscape for rounding corners is as follows: Click the Node Tool (left toolbar) > click on the shape > click on the corner that should be rounded > click Add Corners LPE (top toolbar) > drag the round handles to round the corners.

@BeckerWdf
Copy link
Member Author

  • The spacing of the cut-out lines should be 1.5 px; at the moment it is 1 px.
  • The top right corner is correctly cut, but the corners should also be rounded, as shown in the screenshot:

Both done. Pls. re-review.

@jasmin261098
Copy link
Contributor

Thank you for adding the adjustments.
Looks good to me now, can be merged.

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.

Dual-Tone-Icon-Pack: Platform Icon "save_edit"

3 participants