Skip to content

Try Gutenberg ellipsis menu api#9065

Closed
IreneStr wants to merge 18 commits intotrunkfrom
stories/try-gutenberg-ellipsis-menu-api
Closed

Try Gutenberg ellipsis menu api#9065
IreneStr wants to merge 18 commits intotrunkfrom
stories/try-gutenberg-ellipsis-menu-api

Conversation

@IreneStr
Copy link
Copy Markdown
Contributor

@IreneStr IreneStr commented Feb 27, 2018

DO NOT MERGE

This branch shows how the more menu API from WordPress/gutenberg#4484 can be used.

}

const Icon = () => (
<svg fill="#333" version="1.1" id="Layer_2_1_" xmlns="http://www.w3.org/2000/svg" xmlnsXlink="http://www.w3.org/1999/xlink" viewBox="0 0 841.9 595.3" enableBackground="new 0 0 841.9 595.3" xmlSpace="preserve">
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.

It would be nice to use the attribute we usually add to SVG icons for a11y: role="img" aria-hidden="true" focusable="false"

class IntlProvider extends React.Component {
render() {
if( ! window.Intl ) {
return "Doe us ff niej!";
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.

Uhh.....

@IreneStr IreneStr closed this Sep 6, 2018
@IreneStr IreneStr deleted the stories/try-gutenberg-ellipsis-menu-api branch September 6, 2018 07:48
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.

4 participants