Skip to content

Revise flex-direction property - #44956

Merged
estelle merged 4 commits into
mainfrom
estelle-patch-8
Aug 7, 2026
Merged

Revise flex-direction property#44956
estelle merged 4 commits into
mainfrom
estelle-patch-8

Conversation

@estelle

@estelle estelle commented Jul 29, 2026

Copy link
Copy Markdown
Member
  • removed explanations from syntax section and moved them to new description section
  • changed syntax to standard "keyword" header
  • removed note and moved it to new description section
  • add values intro statement
  • removed descriptions from values and moved them to new description section
  • created a description section
  • updated the links in the a11y section to remove two, update one, and set all to standard mdn way to present external links.

i am not fully sure why were were using main-start and main-end explanations, but i kept them, added cross-start and cross-end, and equated them to inline/block-start/end

@estelle
estelle requested a review from a team as a code owner July 29, 2026 09:25
@estelle
estelle requested review from dipikabh and removed request for a team July 29, 2026 09:25
@github-actions github-actions Bot added Content:CSS Cascading Style Sheets docs size/s [PR only] 6-50 LoC changed labels Jul 29, 2026
Comment thread files/en-us/web/css/reference/properties/flex-direction/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/flex-direction/index.md Outdated
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@github-actions

github-actions Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Preview URLs (1 page)

External URLs (2)

URL: /en-US/docs/Web/CSS/Reference/Properties/flex-direction
Title: flex-direction CSS property

(comment last updated: 2026-08-05 07:29:21)

@dipikabh dipikabh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Great work! Thanks @estelle for the cleanup and the rewrite

Comment thread files/en-us/web/css/reference/properties/flex-direction/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/flex-direction/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/flex-direction/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/flex-direction/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/flex-direction/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/flex-direction/index.md Outdated
Comment on lines -98 to -99
- [Source Order Matters | Adrian Roselli](https://adrianroselli.com/2015/09/source-order-matters.html)
- [MDN Understanding WCAG, Guideline 1.3 explanations](/en-US/docs/Web/Accessibility/Guides/Understanding_WCAG/Perceivable#guideline_1.3_%e2%80%94_create_content_that_can_be_presented_in_different_ways)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not sure if we should remove these two links.
The source order link seems to be directly related to the last sentence about visual vs DOM order

Comment thread files/en-us/web/css/reference/properties/flex-direction/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/flex-direction/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/flex-direction/index.md
Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
@github-actions github-actions Bot added size/m [PR only] 51-500 LoC changed and removed size/s [PR only] 6-50 LoC changed labels Aug 7, 2026
@estelle
estelle merged commit c965bd5 into main Aug 7, 2026
10 checks passed
@estelle
estelle deleted the estelle-patch-8 branch August 7, 2026 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:CSS Cascading Style Sheets docs size/m [PR only] 51-500 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants