Redo/Undo button icons are not customizable

[rmakowiak]

Do you want to request a feature or report a bug?
Bug
What is the current behaviour?
Redo and Undo controls buttons cannot be overwritten with custom icons
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem. GIFs and screenshots are very helpful too.

Editor config that I'm using

<DraftailEditor
        {...otherProps}
        rawContentState={fromHTML(content)}
        onSave={data => onChange(toHTML(data))}
        showUndoControl={{
          description: t('editor.undo'),
          icon: <FontAwesomeIcon icon={faUndo} size="xs" />,
        }.UNDO}
        showRedoControl={{
          description: t('editor.redo'),
          icon: <FontAwesomeIcon icon={faRedo} size="xs" />,
        }}
        inlineStyles={[
          {
            type: INLINE_STYLE.UNDERLINE,
            icon: <FontAwesomeIcon icon={faUnderline} size="xs" />,
            description: t('editor.underline'),
          }
        ]}
      />

Result:

The icon does not appear inside button control.
It works correctly for INLINE or BLOCK styles blocks though.
What is the expected behaviour?
The icon should appear inside button control.
Which versions of Draftail + Draft.js, and which browser / OS are affected by this issue? Did this work in previous versions of Draftail or Draft.js?

^1.0.0, Chrome, MacOS, don't know

[thibaudcolas]

Hey @rmakovyak, thanks for the report!

Looking at the code, it does seem like the icon isn't forwarded to those buttons:

https://github.com/springload/draftail/blob/d90f9e6bb88c0100e8b2e80a178bdd8ad70af7cc/lib/components/ToolbarDefaults.js#L182-L197

This is how it's done for the line break, the implementation should be the same for undo/redo:

https://github.com/springload/draftail/blob/d90f9e6bb88c0100e8b2e80a178bdd8ad70af7cc/lib/components/ToolbarDefaults.js#L161-L163

Would you be up for making a PR to fix this?

[rmakowiak]

hey @thibaudcolas! thanks for quick response. Yes, i'll make a PR with a fix

[thibaudcolas]

Fixed in #196. Sorry it took me so long to get back to this – I’ll try to make a release shortly.

[thibaudcolas]

This is now released in Draftail v1.3.0 – https://www.draftail.org/blog/2019/08/15/draftail-v1-3-0-community-improvements-beyond-wagtail thanks again @rmakovyak for reporting & fixing this.