Skip to content

[wiki] RTL docs update#7199

Merged
cee-chen merged 1 commit intoelastic:mainfrom
cee-chen:wiki/rtl
Sep 19, 2023
Merged

[wiki] RTL docs update#7199
cee-chen merged 1 commit intoelastic:mainfrom
cee-chen:wiki/rtl

Conversation

@cee-chen
Copy link
Copy Markdown
Contributor

@cee-chen cee-chen commented Sep 19, 2023

Summary

Follow up to #7111 (comment)

We discussed as a team using screen from RTL vs destructuring queries from render, and in general decided to clearly document that we prefer render for the EUI codebase.

QA

General checklist

N/A, wiki only

@cee-chen cee-chen added documentation Issues or PRs that only affect documentation - will not need changelog entries skip-changelog Use on PRs to skip changelog requirement (Don't delete - used for automation) labels Sep 19, 2023
@cee-chen cee-chen requested a review from a team September 19, 2023 18:23
@cee-chen cee-chen mentioned this pull request Sep 19, 2023
Merged
9 tasks
1Copenut
1Copenut approved these changes Sep 19, 2023
View reviewed changes
Copy link
Copy Markdown
Contributor

@1Copenut 1Copenut left a comment

Choose a reason for hiding this comment

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

👍 LGTM!

@cee-chen cee-chen merged commit ca5bf9a into elastic:main Sep 19, 2023
@cee-chen cee-chen deleted the wiki/rtl branch September 19, 2023 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@1Copenut 1Copenut 1Copenut approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

documentation Issues or PRs that only affect documentation - will not need changelog entries skip-changelog Use on PRs to skip changelog requirement (Don't delete - used for automation)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@cee-chen @1Copenut