klembot / twinejs

Twine, a tool for telling interactive, nonlinear stories
https://twinery.org
GNU General Public License v3.0
2k stars 295 forks source link

2.4: No indication of default Proofing format #1050

Closed svatas closed 2 years ago

svatas commented 2 years ago

Describe the bug.

After adding a proofing format and selecting it, the "Use to Proof Stories" menu option did not become available. This confused me, because there was no visual indicator that, by adding the format, it had become the current default proofing format. Basically, the current default story and proofing formats should have an indicator on their "cards" to show that they're what will be used by default. A gold "star" icon in the upper-right of their cards or something like that would work.

Steps to reproduce:

  1. Choose menu item Twine - Story format
  2. Click +Add
  3. Insert URL : http://www.maximumverbosity.net/twine/Illume/format.js
  4. Click +Add

Proofing format added but now sign of default one on the screen as 2 exists. By analysis we know, that the latest added is default.

The only possibility to know is to click to the Proofing format and check the status of menu item "Use to Proof Stories". When it is available to click, the Proofing format is not default.

There must be some indication as for Story format - there is word Default but nothing for Proofing format e.g. star icon will solve this.

Expected behavior:

Clear notification of what Story format and what Proofing format is default as o now.

Additional context on this problem.

This is issue created upon description in https://github.com/klembot/twinejs/issues/1009, reported by user @HiEv. As it contains more bug in it, this standalone issue created for better management.

Twine version number

2.4.0 beta-2

Does this problem occur with the web version of Twine or the desktop app?

Desktop app

What operating system does this problem occur on?

Windows

If this problem is occurring with the web version of Twine, what browser does it occur on?

No response

Presubmission checklist

klembot commented 2 years ago

I think what is probably the most apt solution here is to add a "Default" or "Use for Proofing" badge, that's worded sufficiently clearly from the "Proofing" badge we already have.

Screen Shot 2022-02-11 at 12 08 37 AM