Skip to content

Make Plain text as default and fix active index on search #48

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 1 commit into from
Jan 10, 2018

Conversation

lotrien
Copy link
Member

@lotrien lotrien commented Jan 8, 2018

To ensure that syntax is always set was decided to make Plain Text
option the default one. To improve UX for search lang this commit
includes changes on active item, so now it keeps previously clicked value
and gets first from the list if no previous value were provided.

malor
malor previously approved these changes Jan 9, 2018
Copy link
Member

@malor malor left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Do I understand correctly, that this closes both #42 and #45 ? If so, please mention them in the commit message / PR description.

So far <Syntaxes> component is a little bit overcomplicated and requires
thorough reading throughout the code to get its idea. In order to
simplify things and make them more general, let's split the component
into two:

 * general ListBox
 * general ListBoxWithSearch

and encapsulate domain logic (syntaxes) in NewSnippet component.

Co-Authored-By: Olha Kurkaiedova <olya.kurkaedova@gmail.com>
Co-Authored-By: Ihor Kalnytskyi <ihor@kalnytskyi.com>

Closes: #42
Closes: #45
@ikalnytskyi ikalnytskyi merged commit 3844251 into master Jan 10, 2018
@ikalnytskyi ikalnytskyi deleted the syntaxes branch January 10, 2018 22:07
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