Skip to content

Conversation

@yarusome
Copy link
Contributor

@yarusome yarusome commented Apr 2, 2023

Fixes #79.

This PR also aligns the order of values of border-image-repeat to that in the spec.

@yarusome
Copy link
Contributor Author

yarusome commented Apr 9, 2023

@teoli2003 Could I apply for a review from you? I hope this is not too bothering 🙏

@teoli2003
Copy link
Contributor

I tested locally and it doesn't work: space doesn't appear in the drop-down menu. (And if I'm renaming another entry, it does get renamed – so it is not my test that is wrong).

@teoli2003
Copy link
Contributor

Found, we need to increase the height in the css too.

Copy link
Contributor

@teoli2003 teoli2003 left a comment

Choose a reason for hiding this comment

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

👍

@teoli2003 teoli2003 merged commit e68cdcc into mdn:main Apr 9, 2023
@yarusome yarusome deleted the border-image branch April 9, 2023 10:46
@yarusome
Copy link
Contributor Author

yarusome commented Apr 9, 2023

@teoli2003 I just found in the updated generator page that the old values of repeat for the predefined samples got mismatched after this PR. I opened #133 to fix it.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 1, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

border-image generator is missing an option to select border-image-repeat: space;

2 participants