Skip to content

Conversation

@Rusty-UX
Copy link
Contributor

WHY are these changes introduced?

Adds support for missing icons:

  • Bold
  • Italic
  • Underline
  • Text Color
  • Ordered List
  • Indent
  • Outdent
image

Notes

  • Icons have been reviewed by design team
  • Icons have not been tested on polaris website or in admin yet

How to 🎩

🖥 Local development instructions
🗒 General tophatting guidelines
📄 Changelog guidelines

Copy-paste this code in playground/Playground.tsx:
import React from 'react';
import {Page} from '../src';

export function Playground() {
  return (
    <Page title="Playground">
      {/* Add the code you want to test in here */}
    </Page>
  );
}

🎩 checklist

Rusty-UX added 3 commits June 30, 2023 10:44
Adds support for bold, italic, indent, outdent, ordered list, text color, and underline - for usage across Shopify in rich text editors.
@Rusty-UX
Copy link
Contributor Author

Rusty-UX commented Jun 30, 2023

@kyledurand should be good to go, metadata should be correct, and icons are optimized.

FYI I haven't tophatted them on the polaris site, just got stuck in getting development set up. If you are able to test em out that'd be great.

@kyledurand kyledurand requested a review from heyjoethomas June 30, 2023 15:16
@kyledurand kyledurand changed the title Add support for formatting icons Add formatting icons Jun 30, 2023
@kyledurand
Copy link
Member

localhost_3000_icons (2)

@kyledurand kyledurand merged commit 991d9fe into main Jun 30, 2023
@kyledurand kyledurand deleted the formatting-icons branch June 30, 2023 15:53
kyledurand pushed a commit that referenced this pull request Jun 30, 2023
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## @shopify/polaris-icons@7.2.0

### Minor Changes

- [#9581](#9581)
[`991d9fe69`](991d9fe)
Thanks [@Rusty-UX](https://github.com/Rusty-UX)! - Added BoldMajor,
BoldMinor, IndentMajor, IndentMinor, ItalicMajor, ItalicMinor,
OrderedListMajor, OrderedListMinor, OutdentMajor, OutdentMinor,
TextColorMajor, TextColorMinor, UnderlineMajor, UnderlineMinor icons


- [#9580](#9580)
[`75f08f32c`](75f08f3)
Thanks [@kyledurand](https://github.com/kyledurand)! - Added optimize
script

## @shopify/polaris@11.3.1

### Patch Changes

- [#9556](#9556)
[`c72d2f905`](c72d2f9)
Thanks [@aaronccasanova](https://github.com/aaronccasanova)! -
Restructured `Select` override selectors to patch Sass compilation issue


- [#9031](#9031)
[`795ae3782`](795ae37)
Thanks [@danbrady](https://github.com/danbrady)! - Added support for
`Avatar` being presentational

- Updated dependencies
\[[`991d9fe69`](991d9fe),
[`75f08f32c`](75f08f3)]:
    -   @shopify/polaris-icons@7.2.0

## @shopify/polaris-cli@0.2.10



## polaris.shopify.com@0.55.8

### Patch Changes

- Updated dependencies
\[[`c72d2f905`](c72d2f9),
[`795ae3782`](795ae37),
[`991d9fe69`](991d9fe),
[`75f08f32c`](75f08f3)]:
    -   @shopify/polaris@11.3.1
    -   @shopify/polaris-icons@7.2.0

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@sophschneider
Copy link
Contributor

@Rusty-UX @kyledurand just doing the RTE icon mapping now and it looks like the table icon is the only once missing! I'll use AnalyticsTableMinor for now
Screenshot 2023-07-10 at 11 49 33 AM

juzser pushed a commit to juzser/polaris that referenced this pull request Jul 27, 2023
### WHY are these changes introduced?

Adds support for missing icons:
* Bold
* Italic
* Underline
* Text Color 
* Ordered List
* Indent
* Outdent

<img width="962" alt="image"
src="https://github.com/Shopify/polaris/assets/4304867/d0b8ea4d-83d5-4735-8f8c-92c703158baf">

### Notes
* Icons have been reviewed by design team
* Icons have not been tested on polaris website or in admin yet

### How to 🎩

🖥 [Local development
instructions](https://github.com/Shopify/polaris/blob/main/README.md#local-development)
🗒 [General tophatting
guidelines](https://github.com/Shopify/polaris/blob/main/documentation/Tophatting.md)
📄 [Changelog
guidelines](https://github.com/Shopify/polaris/blob/main/.github/CONTRIBUTING.md#changelog)

<details>
<summary>Copy-paste this code in
<code>playground/Playground.tsx</code>:</summary>

```jsx
import React from 'react';
import {Page} from '../src';

export function Playground() {
  return (
    <Page title="Playground">
      {/* Add the code you want to test in here */}
    </Page>
  );
}
```

</details>

### 🎩 checklist

- [ ] 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

---------

Co-authored-by: kyledurand <kyle.durand@shopify.com>
juzser pushed a commit to juzser/polaris that referenced this pull request Jul 27, 2023
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## @shopify/polaris-icons@7.2.0

### Minor Changes

- [Shopify#9581](Shopify#9581)
[`991d9fe69`](Shopify@991d9fe)
Thanks [@Rusty-UX](https://github.com/Rusty-UX)! - Added BoldMajor,
BoldMinor, IndentMajor, IndentMinor, ItalicMajor, ItalicMinor,
OrderedListMajor, OrderedListMinor, OutdentMajor, OutdentMinor,
TextColorMajor, TextColorMinor, UnderlineMajor, UnderlineMinor icons


- [Shopify#9580](Shopify#9580)
[`75f08f32c`](Shopify@75f08f3)
Thanks [@kyledurand](https://github.com/kyledurand)! - Added optimize
script

## @shopify/polaris@11.3.1

### Patch Changes

- [Shopify#9556](Shopify#9556)
[`c72d2f905`](Shopify@c72d2f9)
Thanks [@aaronccasanova](https://github.com/aaronccasanova)! -
Restructured `Select` override selectors to patch Sass compilation issue


- [Shopify#9031](Shopify#9031)
[`795ae3782`](Shopify@795ae37)
Thanks [@danbrady](https://github.com/danbrady)! - Added support for
`Avatar` being presentational

- Updated dependencies
\[[`991d9fe69`](Shopify@991d9fe),
[`75f08f32c`](Shopify@75f08f3)]:
    -   @shopify/polaris-icons@7.2.0

## @shopify/polaris-cli@0.2.10



## polaris.shopify.com@0.55.8

### Patch Changes

- Updated dependencies
\[[`c72d2f905`](Shopify@c72d2f9),
[`795ae3782`](Shopify@795ae37),
[`991d9fe69`](Shopify@991d9fe),
[`75f08f32c`](Shopify@75f08f3)]:
    -   @shopify/polaris@11.3.1
    -   @shopify/polaris-icons@7.2.0

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
AnnaCheba pushed a commit to AnnaCheba/polaris that referenced this pull request Apr 22, 2024
### WHY are these changes introduced?

Adds support for missing icons:
* Bold
* Italic
* Underline
* Text Color 
* Ordered List
* Indent
* Outdent

<img width="962" alt="image"
src="https://github.com/Shopify/polaris/assets/4304867/d0b8ea4d-83d5-4735-8f8c-92c703158baf">

### Notes
* Icons have been reviewed by design team
* Icons have not been tested on polaris website or in admin yet

### How to 🎩

🖥 [Local development
instructions](https://github.com/Shopify/polaris/blob/main/README.md#local-development)
🗒 [General tophatting
guidelines](https://github.com/Shopify/polaris/blob/main/documentation/Tophatting.md)
📄 [Changelog
guidelines](https://github.com/Shopify/polaris/blob/main/.github/CONTRIBUTING.md#changelog)

<details>
<summary>Copy-paste this code in
<code>playground/Playground.tsx</code>:</summary>

```jsx
import React from 'react';
import {Page} from '../src';

export function Playground() {
  return (
    <Page title="Playground">
      {/* Add the code you want to test in here */}
    </Page>
  );
}
```

</details>

### 🎩 checklist

- [ ] 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

---------

Co-authored-by: kyledurand <kyle.durand@shopify.com>
ascherkus pushed a commit to ascherkus/polaris that referenced this pull request Feb 19, 2025
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## @shopify/polaris-icons@7.2.0

### Minor Changes

- [Shopify#9581](Shopify#9581)
[`991d9fe69`](Shopify@a847d9c)
Thanks [@Rusty-UX](https://github.com/Rusty-UX)! - Added BoldMajor,
BoldMinor, IndentMajor, IndentMinor, ItalicMajor, ItalicMinor,
OrderedListMajor, OrderedListMinor, OutdentMajor, OutdentMinor,
TextColorMajor, TextColorMinor, UnderlineMajor, UnderlineMinor icons


- [Shopify#9580](Shopify#9580)
[`75f08f32c`](Shopify@01fb273)
Thanks [@kyledurand](https://github.com/kyledurand)! - Added optimize
script

## @shopify/polaris@11.3.1

### Patch Changes

- [Shopify#9556](Shopify#9556)
[`c72d2f905`](Shopify@0c270b0)
Thanks [@aaronccasanova](https://github.com/aaronccasanova)! -
Restructured `Select` override selectors to patch Sass compilation issue


- [Shopify#9031](Shopify#9031)
[`795ae3782`](Shopify@c68e956)
Thanks [@danbrady](https://github.com/danbrady)! - Added support for
`Avatar` being presentational

- Updated dependencies
\[[`991d9fe69`](Shopify@a847d9c),
[`75f08f32c`](Shopify@01fb273)]:
    -   @shopify/polaris-icons@7.2.0

## @shopify/polaris-cli@0.2.10



## polaris.shopify.com@0.55.8

### Patch Changes

- Updated dependencies
\[[`c72d2f905`](Shopify@0c270b0),
[`795ae3782`](Shopify@c68e956),
[`991d9fe69`](Shopify@a847d9c),
[`75f08f32c`](Shopify@01fb273)]:
    -   @shopify/polaris@11.3.1
    -   @shopify/polaris-icons@7.2.0

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants