Skip to content

Language UI Dropdown new location #1582

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

Merged
merged 8 commits into from
Aug 27, 2020
Merged

Conversation

oruburos
Copy link
Collaborator

@oruburos oruburos commented Aug 26, 2020

Built on #1562

I have verified that this pull request:

  • has no linting errors (npm run lint)
  • is from a uniquely-named feature branch and has been rebased on top of the latest master. (If I was asked to make more changes, I have made sure to rebase onto master then too)
  • is descriptively named and links to an issue number

This PR locates the Language Selector in a new position, and sets a language in defaultprops to pass the test.

@oruburos
Copy link
Collaborator Author

@andrewn Can you have a look at this, please?

Copy link
Member

@andrewn andrewn left a comment

Choose a reason for hiding this comment

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

This is looking really good. Can you remove the en-US default prop and we're good to go!

@oruburos
Copy link
Collaborator Author

@andrewn I got ride of the language entry in default prop, can you have a look, please?

@oruburos
Copy link
Collaborator Author

@andrewn can you have a look at this?

@andrewn andrewn merged commit 3e66c73 into processing:develop Aug 27, 2020
ridait pushed a commit to ridait/p5.js-web-editor that referenced this pull request Aug 28, 2020
* New language selector design
* Adjust authenticated nav to consolidate Account dropdown
* Adding language in NavTest

Co-authored-by: Cassie Tarakajian <ctarakajian@gmail.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