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.

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

use multiline input dialog for annotations #4517

Merged
merged 1 commit into from
Jan 8, 2022

Conversation

ebbit1q
Copy link
Member

@ebbit1q ebbit1q commented Jan 8, 2022

Related Ticket(s)

Short roundup of the initial problem

it's hard to add new lines to annotations

What will change with this Pull Request?

  • changes the dialog from a text one to multiline text
  • pressing enter now makes a new line instead

Screenshots

image

@ZeldaZach ZeldaZach merged commit 59d4e64 into Cockatrice:master Jan 8, 2022
@ebbit1q ebbit1q deleted the multilineannotations branch January 9, 2022 01:19
@MotherofRuins
Copy link

This is nice, thank you, but my preference would be to bind Enter to the OK button and Shift-Enter to add newline in text. The dialog box can helpfully remind people "Please enter the new annotation. Use Shift-Enter for newlines."

@ebbit1q
Copy link
Member Author

ebbit1q commented Jan 28, 2022

that will take more time to make, the built in dialog for multiline text is adequate, isn't it?

@MotherofRuins
Copy link

It's adequate, thanks, but for the most common use case, one line, it would be better if the original interface remained unchanged.

@ebbit1q
Copy link
Member Author

ebbit1q commented Feb 3, 2022

you mean you want it reverted?

@MotherofRuins
Copy link

By original interface, I meant:
Enter = OK should be the same whether in the single-line or multi-line dialog.

@ebbit1q
Copy link
Member Author

ebbit1q commented Mar 17, 2022

this is simply the behavior of the multiline input dialog, if we want to customize that we'd have to implement our own dialog from scratch.

@MotherofRuins
Copy link

MotherofRuins commented Mar 20, 2022

That's too much work, but I imagine people who usually just write a word will complain when this goes live. That's the case with 95% of my annotations. Maybe multiline annotation could be a separate item in the context menus.

@ebbit1q
Copy link
Member Author

ebbit1q commented Mar 25, 2022

I think adding more items to the context menu is the last thing we want, it's already cluttered

@tooomm
Copy link
Member

tooomm commented Mar 25, 2022

Did you consider an extended version of the old dialog?
Instead of using one text input or one multi line input... it could be several text inputs that can be cycled trough with tab if needed (or mouse).
Enter should still submit the data in that case.

Mockup:
Untitled2

A more sophisticated version could add more fields on demand.

@ebbit1q
Copy link
Member Author

ebbit1q commented Mar 25, 2022

this would be a worse version while still making the dialog from scratch...

@tooomm
Copy link
Member

tooomm commented Mar 25, 2022

Probably better to revert this then until a better solution for the initial issue can be proposed?

@tooomm
Copy link
Member

tooomm commented Nov 18, 2023

As expected, people do not like the new behavior of the multi line dialog that much as it makes the default use case worse: #4915

Just that the issue/request itself is proposing a workaround, instead a solution to the root cause. (See also the linked PR #4929)

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.

Line Breaks in Annotations
4 participants