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

Shoes widget not scarpe #307

Merged
merged 3 commits into from
Jul 14, 2023
Merged

Shoes widget not scarpe #307

merged 3 commits into from
Jul 14, 2023

Conversation

noahgibbs
Copy link
Collaborator

@noahgibbs noahgibbs commented Jul 10, 2023

Description

For compatibility and documentation, we're going to want to have portable Shoes stuff under Shoes, not Scarpe. I'm starting on that.

Eventually we'll probably want to match old Shoes classes (e.g. Shoes::Types::Button, not Shoes::Button). But one thing at a time.

Image

This lets us document a bit better:

Screenshot 2023-07-11 at 14 53 34

Checklist

  • Run tests locally
  • Run linter(check for linter errors)

@noahgibbs noahgibbs force-pushed the shoes_widget_not_scarpe branch from e02e231 to 796a172 Compare July 11, 2023 12:18
@noahgibbs noahgibbs marked this pull request as ready for review July 11, 2023 12:19
@noahgibbs noahgibbs force-pushed the shoes_widget_not_scarpe branch from 796a172 to 15f32b5 Compare July 11, 2023 13:54
@noahgibbs noahgibbs force-pushed the shoes_widget_not_scarpe branch from 15f32b5 to b1d1583 Compare July 11, 2023 14:23
@noahgibbs noahgibbs mentioned this pull request Jul 11, 2023
2 tasks
@@ -3,6 +3,8 @@
--protected
--no-private
--template-path docs/yard/template
--tag incompatibility:"Incompatibilities with Shoes"
Copy link
Collaborator

Choose a reason for hiding this comment

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

this is nice..

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I like it, yeah :-) We can add other little categories like this if we want to, but I'm not sure which others we would need.

@Schwad Schwad merged commit 45738cb into main Jul 14, 2023
@noahgibbs noahgibbs deleted the shoes_widget_not_scarpe branch August 1, 2023 09:37
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.

3 participants