"Reload" button for file updated externally shouldn't be a permanent decision

[pintassilgo]

1. Open a file.
2. Edit the file with other editor and save it.
3. Cuda warns the file has changed and ask to reload. Click "Reload".
4. Edit the file with other editor again.
5. Switch back to Cuda.

Expected:
Cuda should warn again that the file has changed.

Actual:
Cuda updates the file automatically, without user consent.

...well, while typing this issue, I discovered this is just a bad default option. Changed it to "ui_notif_confirm": 0, and it was fixed. I suggest to use this as default.

Bonus: in the notification bar there are three buttons: "Reload", "Cancel" and "×". How is user supposed to know the difference between "Cancel" and "×"?

Also, with the option I changed (and suggest to be default), I see no way to tell Cuda to always update one specific file without asking (which should be stored in history session).

So, in summary, my suggestions:

1. Change ui_notif_confirm default to 0.
2. Instead of `"Reload", "Cancel" and "×", the buttons should be these: "Reload", "Always reload this file", "Dismiss" (or "Cancel" or "×" or "Ignore").

[pintassilgo]

Do × mean "No longer warn about updates in this file"? It could be keep, then, but with different wording, like "Never ask for this file".

[Alexey-T]

x means 'Don't reload and disable notifications'.
i can change its title.
changed: "Never ask for this file".

about 'Reload and don't ask': no code/button for this yet, harder to add. maybe it's not needed? does it (or analog) exist in other editors?

[pintassilgo]

Buttons in Kate, for instance: "Enable Auto Reload", "View Difference", "Reload" and "Ignore". So yes, it has "Enable Auto Reload".

In a quick thinking, I believe "Auto reload this file" is way more useful than the "Never ask for this file" that Cuda already has.

Sometimes I have a log file updating every minute and I'm tired of needing to press "Reload" all the time when I want to read it. But this should be file-specific, because I usually don't like auto reload, it's only for very specific files.

[pintassilgo]

Kate screenshot:

[Alexey-T]

ok, agree. button is good. To do for me.

[Alexey-T]

added 4th button 'always reload' on notification-panel.
you can test it.
cudatext.zip

[pintassilgo]

Thank you. It's working as expected. I'm only unsure about the label. It's "Always reload", users would be in doubt if it means always reload every file or always reload this file. "Always reload this file" would be a better title.

I'm also favorable in renaming "Cancel" label. What does it mean to cancel that a file was changed? Maybe to re-save the file to the previous state stored by Cuda? It would be better to be labeled as "Ignore" (like Kate) or "Dismiss". Or a close (×) button. By the way, I believe this cancel/ignore/dismiss/close button should be the last one, following the "rule" that cancel/close buttons are usually placed at the right.

[Alexey-T]

if it means always reload every file or always reload this file.

it means 'for this file' coz panel is inside current file-tab. i don't want to expand the title, need more space for filename.

'cancel'->'ignore'? ok. and i can place it to the end.

[pintassilgo]

it means 'for this file' coz panel is inside current file-tab.

I'm not sure if users would come to this conclusion. Me myself, if it weren't for me talking with you now about the creation of this button, I would be unsure if it means to enable reload for every file or for just this one.

If the issue is in space, you could claim some pixels by doing two changes: 1: replacing "Always" by "Auto", 2: using a × button instead of "Ignore". These two changes would create space for "this file". The × button could use the same rendering as the tab-close button, also the command palette close button.

[Alexey-T]

next beta, all is fixed?
cudatext.zip

[pintassilgo]

Looks perfect, thank you.