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

Comment from IBM: Make border a defined term in 2.4.11 #1373

Closed
mbgower opened this issue Aug 31, 2020 · 3 comments
Closed

Comment from IBM: Make border a defined term in 2.4.11 #1373

mbgower opened this issue Aug 31, 2020 · 3 comments

Comments

@mbgower
Copy link
Contributor

mbgower commented Aug 31, 2020

The current wording of the first bullet of Focus Appearance (Minimum) is:

Minimum area: The focus indication area is greater than or equal to a 1 CSS pixel border of the focused control, or has a thickness of at least 8 CSS pixels along the shortest side of the element.

The concept of "border" must be understood for users to properly interpret the resulting formula. Since the calculation itself is not part of the normative content, we recommend defining the term to make this formula clearer and to reduce translation issues. A non-normative note already exists, which we have modified as a definition.

A border is the perimeter of the control and defines its shape. Standard formulae exist for calculating the perimeters of rectangles, circles and other geometric shapes. A border is not a reference to the CSS property of the same name.

Potentially the definition could specifically include the formula for rectangles and circles.

@mbgower mbgower changed the title Comment from IBM: Make border a defined turn in 2.4.11 Comment from IBM: Make border a defined term in 2.4.11 Aug 31, 2020
@36degrees
Copy link
Member

I think this overlaps quite a lot with #1324, as border not being clearly defined leads to the ambiguity mentioned there.

@alastc
Copy link
Contributor

alastc commented Oct 22, 2020

@mbgower - It looks like your re-write in #1403 included a definition of perimeter, so would close this issue?

@alastc
Copy link
Contributor

alastc commented Jan 6, 2021

We have perimeter as a defined term now, so closing this issue.

@alastc alastc closed this as completed Jan 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants