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

Update the Campaign Properties -> Token Properties Tab #1238

Closed
cwisniew opened this issue Feb 4, 2020 · 7 comments
Closed

Update the Campaign Properties -> Token Properties Tab #1238

cwisniew opened this issue Feb 4, 2020 · 7 comments
Labels
feature Adding functionality that adds value Hacktoberfest up for grabs Minimal context and are well-suited for new contributors

Comments

@cwisniew
Copy link
Member

cwisniew commented Feb 4, 2020

Is your feature request related to a problem? Please describe.
The current Token Properties Tab in the Campaign properties is pretty primitive requiring a somewhat esoteric text format to define properties and where they are displayed.
image
image

Describe the solution you'd like
This should be something that is easier to edit, something like a table with the property name, the display on stat sheet properties (either as set of check boxes or a drop down box), and default value.

Additional context
This could be combined with an update of Sight and Light tabs for Campaign Properties Dialog (issues #1239 & #1240)

@cwisniew cwisniew added the up for grabs Minimal context and are well-suited for new contributors label Feb 4, 2020
@cwisniew cwisniew added the feature Adding functionality that adds value label Feb 4, 2020
@JamzTheMan
Copy link
Member

Can we use FX to replace these "dialogs"?

@cwisniew
Copy link
Member Author

cwisniew commented Feb 4, 2020

Yes but campaign properties would have to be made thread safe. I am not near a computer at the moment but I can check how big a task that would be in the morning

@cwisniew
Copy link
Member Author

cwisniew commented Feb 5, 2020

See #1247 for discussion about thread safety changes required if JavaFX is to be used for these dialogs

@Illydth
Copy link

Illydth commented Jun 27, 2020

As an additional "Quality of Life", can we get a check upon hitting "OK" that there aren't pending changes (at least) in the Token Properties tab? I cannot begin to explain how many lost minutes to hours of work have gone into modifying properties and then hitting "OK" to save the changes only to realize that OK is actually Cancel along with cancel and that Update is really OK. (yea, I know that was confusing, intentionally so).

At the end of the day the standard GUI Consensus is that there are 2 buttons: OK and Cancel. OK saves changes, Cancel Aborts Changes. Interfaces do often have an "update" button, but that acts as a "Update Current Settings and Stay in Place" vs. OK which is "Update Settings and Close Window."

It's not that clicking an "update" button is so immeasurably harder to manage, it's positioning that's the problem. Every Modern GUI Interface we use now adays has a very common place to click to save work and exit, and that common location is a bit of a "social contract". Our hands/muscle memory is tuned to complete changes and click "Ok" in the bottom of the window on the left of the two buttons.

With "Ok" Acting only like "Close Dialog" (not that a name change would fix the problem, it wouldn't), that somewhat breaks the social contract. "OK" in the case where you haven't clicked "update" first, acts as if you'd pressed CANCEL...which is why I, and I have to assume many others, especially newer users, lose work in campaign properties so frequently.

At minimum, "Ok" should pop a dialog up that says "You have open unsaved work, do you wish to continue?" (a dialog that is very reasonable to be able to turn off in preferences for those who are smarter than I am and can handle clicking something other than "ok" to save their work.)

@Azhrei
Copy link
Member

Azhrei commented Jun 27, 2020

The purpose of this issue is to completely replace the UI. The update/ok/cancel would then go away.

But I feel you’re pain — It’s happened to me, too. In fact, just switching from one property type to another requires hitting Update and, as you say, there’s no warning that switching will lose data.

@Tdue21
Copy link
Contributor

Tdue21 commented Jan 13, 2024

@cwisniew Isn't this issue closed with 1.14?

@cwisniew
Copy link
Member Author

@cwisniew Isn't this issue closed with 1.14?

yep! Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Adding functionality that adds value Hacktoberfest up for grabs Minimal context and are well-suited for new contributors
Projects
None yet
Development

No branches or pull requests

5 participants