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

Fixed typos, ordering and icons #10901

Merged
merged 3 commits into from
Oct 5, 2023
Merged

Conversation

tjonx
Copy link
Contributor

@tjonx tjonx commented Oct 5, 2023

WHY are these changes introduced?

  • Fixes alignment issues between pages
  • Fixes some ordering issues in some sections
  • Adds icons for Motion pages
  • Fixes some typos

WHAT is this pull request doing?

Before After
image image
image image
image image
image image

🎩 checklist

@tjonx tjonx self-assigned this Oct 5, 2023
@tjonx tjonx requested a review from a team October 5, 2023 17:11
@tjonx tjonx added #gsd:36550 🤖Skip Changelog Causes CI to ignore changelog update check. labels Oct 5, 2023
@tjonx tjonx merged commit 953ebf2 into next Oct 5, 2023
7 checks passed
@tjonx tjonx deleted the tjonx/adjust-markdown-docs-release branch October 5, 2023 19:11
mrcthms pushed a commit that referenced this pull request Oct 12, 2023
<!--
  ☝️How to write a good PR title:
- Prefix it with [ComponentName] (if applicable), for example: [Button]
  - Start with a verb, for example: Add, Delete, Improve, Fix…
  - Give as much context as necessary and as little as possible
  - Prefix it with [WIP] while it’s a work in progress
-->

### WHY are these changes introduced?

- Fixes alignment issues between pages
- Fixes some ordering issues in some sections
- Adds icons for Motion pages
- Fixes some typos

<!--
  Context about the problem that’s being addressed.
-->

### WHAT is this pull request doing?

| Before | After |
|--------|-------|
|
![image](https://github.com/Shopify/polaris/assets/67433661/b5a55249-b0b5-4e4c-bd46-c93af5fc3897)
|
![image](https://github.com/Shopify/polaris/assets/67433661/1dd2f8e7-54d3-49bf-940b-ac6e0e5dc0ca)
|
|
![image](https://github.com/Shopify/polaris/assets/67433661/ab6a82e3-16b6-4f73-89e7-c233fa333406)
|
![image](https://github.com/Shopify/polaris/assets/67433661/de54cb8e-e8c6-4c0a-833d-b1329ea69b7c)
|
|
![image](https://github.com/Shopify/polaris/assets/67433661/ab02e689-99b4-49d7-8a6c-7f0e43cbd061)
|
![image](https://github.com/Shopify/polaris/assets/67433661/0b6bc969-39e5-4703-971b-c94cfac05094)
|
|
![image](https://github.com/Shopify/polaris/assets/67433661/8e67dedc-503e-4e18-936c-362e037c580d)
|
![image](https://github.com/Shopify/polaris/assets/67433661/74f6ce7e-55c7-4c2b-8fa2-9ea7dd96583c)
|


### 🎩 checklist

- [x] Tested on
[mobile](https://github.com/Shopify/polaris/blob/main/documentation/Tophatting.md#cross-browser-testing)
- [ ] Tested on [multiple
browsers](https://help.shopify.com/en/manual/shopify-admin/supported-browsers)
- [ ] Tested for
[accessibility](https://github.com/Shopify/polaris/blob/main/documentation/Accessibility%20testing.md)
- [ ] Updated the component's `README.md` with documentation changes
- [ ] [Tophatted
documentation](https://github.com/Shopify/polaris/blob/main/documentation/Tophatting%20documentation.md)
changes in the style guide
mrcthms pushed a commit that referenced this pull request Oct 12, 2023
<!--
  ☝️How to write a good PR title:
- Prefix it with [ComponentName] (if applicable), for example: [Button]
  - Start with a verb, for example: Add, Delete, Improve, Fix…
  - Give as much context as necessary and as little as possible
  - Prefix it with [WIP] while it’s a work in progress
-->

### WHY are these changes introduced?

- Fixes alignment issues between pages
- Fixes some ordering issues in some sections
- Adds icons for Motion pages
- Fixes some typos

<!--
  Context about the problem that’s being addressed.
-->

### WHAT is this pull request doing?

| Before | After |
|--------|-------|
|
![image](https://github.com/Shopify/polaris/assets/67433661/b5a55249-b0b5-4e4c-bd46-c93af5fc3897)
|
![image](https://github.com/Shopify/polaris/assets/67433661/1dd2f8e7-54d3-49bf-940b-ac6e0e5dc0ca)
|
|
![image](https://github.com/Shopify/polaris/assets/67433661/ab6a82e3-16b6-4f73-89e7-c233fa333406)
|
![image](https://github.com/Shopify/polaris/assets/67433661/de54cb8e-e8c6-4c0a-833d-b1329ea69b7c)
|
|
![image](https://github.com/Shopify/polaris/assets/67433661/ab02e689-99b4-49d7-8a6c-7f0e43cbd061)
|
![image](https://github.com/Shopify/polaris/assets/67433661/0b6bc969-39e5-4703-971b-c94cfac05094)
|
|
![image](https://github.com/Shopify/polaris/assets/67433661/8e67dedc-503e-4e18-936c-362e037c580d)
|
![image](https://github.com/Shopify/polaris/assets/67433661/74f6ce7e-55c7-4c2b-8fa2-9ea7dd96583c)
|


### 🎩 checklist

- [x] Tested on
[mobile](https://github.com/Shopify/polaris/blob/main/documentation/Tophatting.md#cross-browser-testing)
- [ ] Tested on [multiple
browsers](https://help.shopify.com/en/manual/shopify-admin/supported-browsers)
- [ ] Tested for
[accessibility](https://github.com/Shopify/polaris/blob/main/documentation/Accessibility%20testing.md)
- [ ] Updated the component's `README.md` with documentation changes
- [ ] [Tophatted
documentation](https://github.com/Shopify/polaris/blob/main/documentation/Tophatting%20documentation.md)
changes in the style guide
AnnaCheba pushed a commit to AnnaCheba/polaris that referenced this pull request Apr 22, 2024
<!--
  ☝️How to write a good PR title:
- Prefix it with [ComponentName] (if applicable), for example: [Button]
  - Start with a verb, for example: Add, Delete, Improve, Fix…
  - Give as much context as necessary and as little as possible
  - Prefix it with [WIP] while it’s a work in progress
-->

### WHY are these changes introduced?

- Fixes alignment issues between pages
- Fixes some ordering issues in some sections
- Adds icons for Motion pages
- Fixes some typos

<!--
  Context about the problem that’s being addressed.
-->

### WHAT is this pull request doing?

| Before | After |
|--------|-------|
|
![image](https://github.com/Shopify/polaris/assets/67433661/b5a55249-b0b5-4e4c-bd46-c93af5fc3897)
|
![image](https://github.com/Shopify/polaris/assets/67433661/1dd2f8e7-54d3-49bf-940b-ac6e0e5dc0ca)
|
|
![image](https://github.com/Shopify/polaris/assets/67433661/ab6a82e3-16b6-4f73-89e7-c233fa333406)
|
![image](https://github.com/Shopify/polaris/assets/67433661/de54cb8e-e8c6-4c0a-833d-b1329ea69b7c)
|
|
![image](https://github.com/Shopify/polaris/assets/67433661/ab02e689-99b4-49d7-8a6c-7f0e43cbd061)
|
![image](https://github.com/Shopify/polaris/assets/67433661/0b6bc969-39e5-4703-971b-c94cfac05094)
|
|
![image](https://github.com/Shopify/polaris/assets/67433661/8e67dedc-503e-4e18-936c-362e037c580d)
|
![image](https://github.com/Shopify/polaris/assets/67433661/74f6ce7e-55c7-4c2b-8fa2-9ea7dd96583c)
|


### 🎩 checklist

- [x] Tested on
[mobile](https://github.com/Shopify/polaris/blob/main/documentation/Tophatting.md#cross-browser-testing)
- [ ] Tested on [multiple
browsers](https://help.shopify.com/en/manual/shopify-admin/supported-browsers)
- [ ] Tested for
[accessibility](https://github.com/Shopify/polaris/blob/main/documentation/Accessibility%20testing.md)
- [ ] Updated the component's `README.md` with documentation changes
- [ ] [Tophatted
documentation](https://github.com/Shopify/polaris/blob/main/documentation/Tophatting%20documentation.md)
changes in the style guide
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖Skip Changelog Causes CI to ignore changelog update check.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants