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

Added possible values for table frame attribute #28399

Closed
wants to merge 9 commits into from

Conversation

EzioMercer
Copy link

@EzioMercer EzioMercer commented Aug 4, 2023

Added possible values for table frame attribute

Possible values were missed so I added them

I'm Frontend teacher and I always try to use MDN Web Docs to show my students any information. When I tried to find possible values of frame attribute I had to go to other web pages. I think it will be great if all need information will be in MDN Web Docs and my students will see that they always can use this documentation

Docs where I got the information from

@EzioMercer EzioMercer requested a review from a team as a code owner August 4, 2023 17:17
@EzioMercer EzioMercer requested review from estelle and removed request for a team August 4, 2023 17:17
@github-actions github-actions bot added the Content:HTML Hypertext Markup Language docs label Aug 4, 2023
@EzioMercer EzioMercer changed the title Added possible values for frame attribute Added possible values for table frame attribute Aug 4, 2023
@estelle
Copy link
Member

estelle commented Aug 4, 2023

Thank you for the contribution. The frame attribute of the <table> element is deprecated. The spec we follow on MDN is the WHATWG HTML living standard. https://html.spec.whatwg.org/multipage/tables.html#the-table-element. (Your link is HTML 4.01). Tables only have global attributes. styling should be done with CSS (separation of concerns), so all appearance related attributes are deprecated.

Here is content that may be helpful for your students:

Styling tables:
https://developer.mozilla.org/en-US/docs/Learn/HTML/Tables
https://web.dev/learn/html/tables/

Marking up tables:
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/table#see_also
https://developer.mozilla.org/en-US/docs/Learn/CSS/Building_blocks/Styling_tables

@estelle estelle closed this Aug 4, 2023
@EzioMercer
Copy link
Author

EzioMercer commented Aug 20, 2023

@estelle Thanks for answer but I didn't understand your point. The rules attribute is also deprecated but you specify the possible values for this attribute. Why not do it for the frame attribute? What is difference between these two deprecated attributes?

@estelle
Copy link
Member

estelle commented Aug 21, 2023

you have a very valid point. We should create an issue to determine whether we should be adding all the deprecated enumerated values that are missing (your PR), or removing the deprecated values that should not be used in favor of using CSS. The CSS comments are there, which is good, but unclear why we are including enumerated values for some deprecated attributes but not others.

@estelle
Copy link
Member

estelle commented Sep 19, 2023

Let's also change rules to read:

This enumerated attribute defines where rules, i.e. lines, should appear in a table. Lines can be set to appear between `rows`, `cols` (for columns), or `groups` (between  `<thead>`, `<tbody>`, and `<tfoot>` elements, and between `<col>` and `<colgroup>` elements), or `all` or `none` of the above.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 19, 2023

Preview URLs

(comment last updated: 2024-01-12 13:26:00)

@EzioMercer
Copy link
Author

@estelle Can you please help me? I can't understand how to update the PR preview page. I made changes which you asked but can't see the result

Copy link
Member

@estelle estelle left a comment

Choose a reason for hiding this comment

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

Suggested some edits

Comment on lines +122 to +132
- : This enumerated attribute defines which side of the frame surrounding the table must be displayed. It can have the following values:

- `void`, which indicates that no sides will be displayed; it is the default value;
- `above`, which indicates that the top side only will be displayed;
- `below`, which indicates that the bottom side only will be displayed;
- `hsides`, which indicates that the top and bottom sides only will be displayed;
- `vsides`, which indicates that the right and left sides only will be displayed;
- `lhs`, which indicates that the left-hand side only will be displayed;
- `rhs`, which indicates that the right-hand side only will be displayed;
- `box`, which indicates that the all four sides will be displayed;
- `border`, which indicates that the all four sides will be displayed; it is same as `box`;
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- : This enumerated attribute defines which side of the frame surrounding the table must be displayed. It can have the following values:
- `void`, which indicates that no sides will be displayed; it is the default value;
- `above`, which indicates that the top side only will be displayed;
- `below`, which indicates that the bottom side only will be displayed;
- `hsides`, which indicates that the top and bottom sides only will be displayed;
- `vsides`, which indicates that the right and left sides only will be displayed;
- `lhs`, which indicates that the left-hand side only will be displayed;
- `rhs`, which indicates that the right-hand side only will be displayed;
- `box`, which indicates that the all four sides will be displayed;
- `border`, which indicates that the all four sides will be displayed; it is same as `box`;
- : This enumerated attribute specifies which outer sides of the table should have borders drawn. The default `void` means no borders. The `above`, `below`, `lhs` (left-hand side), and `rhs` (right-hand side) lead to a border on a single side. Two sides will have borders with `hsides` (horizontal sides), `vsides` (vertical sides). All four sides will have a border if `box` or `border` is set. The CSS {{cssxref("border-style")}} property, set on the table, supersedes borders set by this HTML attribute.

- `rows`, which will cause the rules to be displayed between rows;
- `cols`, which will cause the rules to be displayed between columns;
- `all`, which will cause the rules to be displayed between rows and columns.
- : This enumerated attribute defines where rules, i.e. lines, should appear in a table. Lines can be set to appear between `rows`, `cols` (for columns), or `groups` (between `<thead>`, `<tbody>`, and `<tfoot>` elements, and between `<col>` and `<colgroup>` elements), or `all` or `none` of the above.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- : This enumerated attribute defines where rules, i.e. lines, should appear in a table. Lines can be set to appear between `rows`, `cols` (for columns), or `groups` (between `<thead>`, `<tbody>`, and `<tfoot>` elements, and between `<col>` and `<colgroup>` elements), or `all` or `none` of the above.
- : This enumerated attribute defines where rules, i.e. lines, should appear in a table. Lines can be set to appear between `rows`, `cols` (for columns), or `groups` (between `<thead>`, `<tbody>`, and `<tfoot>` elements, and between `<col>` and `<colgroup>` elements), or `all` or `none` of the above. CSS {{cssxref("border-style")}} supersedes borders set by this attribute.

@estelle
Copy link
Member

estelle commented Oct 12, 2023

@estelle Can you please help me? I can't understand how to update the PR preview page. I made changes which you asked but can't see the result

https://pr28399.content.dev.mdn.mozit.cloud/en-US/docs/Web/HTML/Element/table

the preview feature was down for a few days, so maybe it wasn't showing, but generally search in this page for "Preview URLs" and it should direct you to the github action creating a preview link

@EzioMercer
Copy link
Author

@estelle Yes I know about that links but I can't understand how to update them and/or when do they updates automatically?

As you can see here I already made changes for rules attribute but the preview link still shows old version of this page

image

So I'm a little confused. I thought that this page will update after each commit

@bsmth
Copy link
Member

bsmth commented Jan 12, 2024

Hi @EzioMercer - there are some review comments above waiting on you. Do you plan to come back to this one?

@bsmth bsmth added the awaiting response Awaiting for author to address review/feedback label Jan 12, 2024
@EzioMercer
Copy link
Author

Hi @bsmth! Of course I will, I'm waiting the response to my last question. I can't update the preview link to see how my last changes look like

@bsmth
Copy link
Member

bsmth commented Jan 12, 2024

Hi @bsmth! Of course I will, I'm waiting the response to my last question. I can't update the preview link to see how my last changes look like

I understand 👍🏻. You can check this sticky comment: #28399 (comment) - It should have the latest changes from this branch. I can see your proposed changes there as expected.

@bsmth
Copy link
Member

bsmth commented Jan 18, 2024

It looks like this PR has stalled. What's next steps here @EzioMercer and @estelle?

@github-actions github-actions bot added the merge conflicts 🚧 [PR only] label Jan 25, 2024
Copy link
Contributor

This pull request has merge conflicts that must be resolved before it can be merged.

@bsmth
Copy link
Member

bsmth commented Feb 6, 2024

Hi both 👋🏻 I am going to close this one as it's stalled with conflicts & suggestions etc., additionally with the context of adding more info to deprecated features when there are suggested alternatives, I don't think we need to keep improving this section. Thank you! :)

@bsmth bsmth closed this Feb 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting response Awaiting for author to address review/feedback Content:HTML Hypertext Markup Language docs merge conflicts 🚧 [PR only]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants