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

Border module includes invalid outline properties #223

Open
nsaunders opened this issue Feb 10, 2022 · 4 comments
Open

Border module includes invalid outline properties #223

nsaunders opened this issue Feb 10, 2022 · 4 comments

Comments

@nsaunders
Copy link
Contributor

MDN's border-style documentation, as an example, shows "constituent properties" that allow a unique border to be applied to each side of an element. However, this does not seem to be the case for outlines: Notice, for example, how outline-style has no constituent properties listed. Indeed, my own testing suggests that outline properties can't be assigned to each side of an element independently: For example, outline-top-color yields an "unsupported property" warning in Safari:

image

@turion
Copy link
Collaborator

turion commented Feb 15, 2022

Ok, so outlineTopColor and similar functions should be removed?

@nsaunders
Copy link
Contributor Author

@turion that is correct. These are the only outline functions that produce valid CSS:

  • outline
  • outlineColor
  • outlineStyle
  • outlineWidth
  • outlineOffset

So these should be removed:

  • outlineTop
  • outlineRight
  • outlineBottom
  • outlineLeft
  • outlineColor4
  • outlineTopColor
  • outlineRightColor
  • outlineBottomColor
  • outlineLeftColor
  • outlineStyle4
  • outlineTopStyle
  • outlineRightStyle
  • outlineBottomStyle
  • outlineLeftStyle
  • outlineWidth4
  • outlineTopWidth
  • outlineRightWidth
  • outlineBottomWidth
  • outlineLeftWidth

Copy link

This issue has not seen any activity in a long time. If no further activity occurs, it will be closed after ten weeks.

@turion
Copy link
Collaborator

turion commented Oct 25, 2024

PRs welcome to this issue :)

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

No branches or pull requests

2 participants