Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Feature/246 input clear button #279

Merged
merged 3 commits into from
Apr 30, 2021
Merged

Feature/246 input clear button #279

merged 3 commits into from
Apr 30, 2021

Conversation

caebr
Copy link
Collaborator

@caebr caebr commented Apr 26, 2021

No description provided.

@caebr caebr requested a review from hupf April 26, 2021 10:00
@caebr caebr self-assigned this Apr 26, 2021
hupf
hupf approved these changes Apr 26, 2021
View reviewed changes
Copy link
Collaborator

@hupf hupf left a comment

Choose a reason for hiding this comment

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

Ich konnte irgendwie keine Tokens mehr lösen, deshalb nicht im UI getestet. Im Code sieht es gut aus. Die Frage ist, ob du nicht gleich eine Komponenten dafür machen möchtest, da es doch an ein paar Orten vorkommt (ResettableInput oder ähnlich).

@caebr
Copy link
Collaborator Author

caebr commented Apr 26, 2021

Ich konnte irgendwie keine Tokens mehr lösen, deshalb nicht im UI getestet. Im Code sieht es gut aus. Die Frage ist, ob du nicht gleich eine Komponenten dafür machen möchtest, da es doch an ein paar Orten vorkommt (ResettableInput oder ähnlich).

Ah, ja - das Backend ist heute Nachmittag offline. Deshalb kannst du keine Tokens lösen.
Das mit der Komponente hatte ich mir auch überlegt, aber aus Zeitgründen gelassen. Ich schaus mir aber nochmals kurz an.

@caebr caebr merged commit 986c918 into master Apr 30, 2021
@mfehlmann mfehlmann deleted the feature/246-input-clear-button branch April 21, 2022 11:42
@mfehlmann mfehlmann restored the feature/246-input-clear-button branch April 21, 2022 11:42
@mfehlmann mfehlmann deleted the feature/246-input-clear-button branch April 21, 2022 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hupf hupf hupf approved these changes

Assignees

@caebr caebr

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@caebr @hupf