View Issue Details

IDProjectCategoryView StatusLast Update
0002628Far ManagerEditorpublic2014-04-09 06:34
Reporterfml2 Assigned ToDrKnS  
PrioritynormalSeverityminorReproducibilityalways
Status assignedResolutionopen 
Product Version3.0 
Summary0002628: Предупреждение с непонятными следствиями выбора
DescriptionЕсли в редакторе при изменении кодировки были "потеряны" некоторые символы, то при последующей попытке сохранить файл появляется следующее предупреждение (английский вариант):

╔════════════════ Warning ═════════════════╗
║ During file editing some data were lost. ║
║ It is not recommended to save this file. ║
╟──────────────────────────────────────────╢
║ { OK } [ Cancel ] ║
╚══════════════════════════════════════════╝

Мне кажется, текст сформулирован так, что непонятно, что произойдет, если нажать ОК. Мне кажется, в текст нужно бы добавить что-то типа "Do you still wnt to save the file?".
TagsNo tags attached.
Build

Activities

fml2

2014-03-22 19:09

reporter   bugnote:0012019

wnt --> want

fml2

2014-03-23 17:15

reporter   bugnote:0012020

В русском варианте текст тоже сформулирован так, что непонятно, что произойдет, если выбрать "ОК":

╔═══════════════════════ Предупреждение ═══════════════════════╗
║ Во время редактирования файла некоторые данные были утеряны. ║
║ Сохранять файл не рекомендуется. ║
╟──────────────────────────────────────────────────────────────╢
║ { OK } [ Отмена ] ║
╚══════════════════════════════════════════════════════════════╝

JohnDoe

2014-03-24 10:31

updater   bugnote:0012024

Можно кнопку OK переименовать в Сохранить.

fml2

2014-03-25 10:00

reporter   bugnote:0012030

> Можно кнопку OK переименовать в Сохранить.

Мне такая идея нравится.

fml2

2014-03-29 22:24

reporter   bugnote:0012059

Вот, сделал патчик. Сгодится?

fml2

2014-03-29 22:24

reporter  

editor-save.diff (1,090 bytes)   
Index: unicode_far/farlang.templ.m4
===================================================================
--- unicode_far/farlang.templ.m4	(revision 11584)
+++ unicode_far/farlang.templ.m4	(working copy)
@@ -7581,6 +7581,16 @@
 "No se recomienda guardar este archivo."
 "Neodporúčame uložiť tento súbor."
 
+MEditorSave
+"Сохранить"
+"Save"
+"Uložit"
+"Speichern"
+"Mentés"
+"Zapisz"
+"Guardar"
+"Uložiť"
+
 MEditorSaveCPWarnShow
 "Показать"
 "Show"
Index: unicode_far/fileedit.cpp
===================================================================
--- unicode_far/fileedit.cpp	(revision 11584)
+++ unicode_far/fileedit.cpp	(working copy)
@@ -1599,7 +1599,7 @@
 	BadConversion = !GetStr.IsConversionValid();
 	if (BadConversion)
 	{
-		Message(MSG_WARNING,1,MSG(MWarning),MSG(MEditorLoadCPWarn1),MSG(MEditorLoadCPWarn2),MSG(MEditorSaveNotRecommended),MSG(MOk));
+		Message(MSG_WARNING,1,MSG(MWarning),MSG(MEditorLoadCPWarn1),MSG(MEditorLoadCPWarn2),MSG(MEditorSaveNotRecommended),MSG(MEditorSave));
 	}
 
 	if (LastLineCR||!m_editor->NumLastLine)
editor-save.diff (1,090 bytes)   

DrKnS

2014-03-30 09:25

administrator   bugnote:0012060

3846.

Будьте внимательны, ваш патчик правит совсем не то что нужно.

fml2

2014-03-31 08:15

reporter   bugnote:0012065

Виноват! Спасибо, что подправили. Теперь уже лучше.

Еще одно замечание по поводу этого диалога: обычно в диалогах, предупреждающих об "опасных" действиях, фокус изначально находится на "неопасной" кнопке (например, "отмена"). А здесь фоку с сразу на "Сохранить". Не лучше ли это поменять?

DrKnS

2014-03-31 20:20

administrator   bugnote:0012067

"обычно"? Например?

fml2

2014-03-31 21:26

reporter   bugnote:0012068

Чтобы далеко не ходить за примером, приведу хотя бы диалог (из того же фаровского редактора), который показывается тогда, когда не удается корректно поменять кодировку:

╔═══════════════════ Warning ═══════════════════╗
║ Editor contains characters, which cannot be ║
║ correctly translated using selected codepage. ║
║ Continue? ║
╟───────────────────────────────────────────────╢
║ { Cancel } [ Show ] [ OK ] ║
╚═══════════════════════════════════════════════╝

Можно легко привести примеры и из других программ.

DrKnS

2014-04-06 15:43

administrator   bugnote:0012116

Один диалог на весь фар - это как-то не тянет на "обычно".

fml2

2014-04-06 16:51

reporter   bugnote:0012117

Это был не "один диалог на весь Фар", а всего лишь один пример. Хотите еще? Пожалуйста: при попытке закрыть редактор с несохраненными изменениями диалог выглядит так:

╔═══════════ Editor ════════════╗
║ File has been modified. Save? ║
╟───────────────────────────────╢
║ { Yes } [ No ] [ Cancel ] ║
╚═══════════════════════════════╝

Т.е. опять действие "по умолчанию" -- безопасное, т.е. без потери данных.

Мне даже странно, что такому опытному человеку как вы, это надо объяснять. Или вы просто хотите попрепираться?

DrKnS

2014-04-06 17:14

administrator   bugnote:0012118

Цель подтверждений - внезапно, подтверждать выполнение операций, а не активно препятствовать их выполнению.
Перетасовывание кнопок - это как раз второй случай.
На уровне рефлексов ожидается, что ок - слева, отмена - справа, enter - согласен, esc - не согласен.
А тут не только текст прочитай, но и разберись, как это предупреждение закрыть.

Удаление файлов - не менее деструктивная операция, давайте еще там кнопки тасовать, ну или сразу капчу.

И да, мне больше делать нечего, только с вами препираться.

fml2

2014-04-06 18:58

reporter   bugnote:0012121

Last edited: 2014-04-06 19:01

Мне кажется, это не подтверждения, а предупреждения. И их цель -- предупредить пользователя о возможной потере данных. В этих случаях слепое нажатие "Ввод" в диалоге должно приводить к "не-потере" данных

Я согласен, что удаление файлов в этом слмысле -- это кажущееся исключение. Но это только так кажется. При удалении удаление файлов было явным желанием пользователя. А при перекодировке потеря информации -- это, скорее, побочный эффект.

JohnDoe

2014-04-08 10:49

updater   bugnote:0012166

>Мне кажется, это не подтверждения, а предупреждения.
В данном случае это будет второе предупреждение.
Т.е. пользователь один раз (преодолев все препятствия) согласился на деструктивное преобразование. И теперь хочет-таки сохранить этот файл.

Я совершенно не уверен что тут нужно менять действие по умолчанию.
И уж точно не в случае когда мы сохраняем файл под новым именем (Shift-F2).

fml2

2014-04-09 06:34

reporter   bugnote:0012177

По поводу Shift+F2 согласен на 100%, т.к. при этом никакой потери данных не будет. По поводу 2-го предупреждения -- тоже на 100% (глупо было бы отрицать, что это 2-е предупреждение :-), но по поводу того, что "Сохранить" должно быть действием по умолчанию, я все же не совсем согласен. Все-таки это побочный эффект. А о том, что было раньше (т.е. при первом предупреждении), пользователь мог уже и забыть.

Еще один аргумент в мою пользу: в тексте окна написано "Сохранять файл не рекомендуется" -- и тут же предлагается по умолчанию все же сохранить его. Противоречиво получается.

Issue History

Date Modified Username Field Change
2014-03-22 19:07 fml2 New Issue
2014-03-22 19:09 fml2 Note Added: 0012019
2014-03-23 17:15 fml2 Note Added: 0012020
2014-03-24 10:31 JohnDoe Note Added: 0012024
2014-03-25 10:00 fml2 Note Added: 0012030
2014-03-29 22:24 fml2 Note Added: 0012059
2014-03-29 22:24 fml2 File Added: editor-save.diff
2014-03-30 09:25 DrKnS Note Added: 0012060
2014-03-30 09:25 DrKnS Assigned To => DrKnS
2014-03-30 09:25 DrKnS Status new => feedback
2014-03-31 08:15 fml2 Note Added: 0012065
2014-03-31 08:15 fml2 Status feedback => assigned
2014-03-31 20:20 DrKnS Note Added: 0012067
2014-03-31 21:26 fml2 Note Added: 0012068
2014-04-06 15:43 DrKnS Note Added: 0012116
2014-04-06 16:51 fml2 Note Added: 0012117
2014-04-06 17:14 DrKnS Note Added: 0012118
2014-04-06 18:58 fml2 Note Added: 0012121
2014-04-06 19:01 fml2 Note Edited: 0012121
2014-04-08 10:49 JohnDoe Note Added: 0012166
2014-04-09 06:34 fml2 Note Added: 0012177