View Issue Details

IDProjectCategoryView StatusLast Update
0000647WishesDialogpublic2013-02-04 00:30
Reporteralexmitin Assigned ToDrKnS  
PrioritynormalSeverityfeatureReproducibilityalways
Status closedResolutionfixed 
Fixed in Version2.0 
Summary0000647: Button styles in the dialogs
DescriptionThe dialog buttons should have a space separator between buttons.

Currently some dialogs have a space between buttons - for example, "Apply command" dialog (Ctrl+G). The other dialogs do not have a space between buttons - for example, "Delete" dialog (F8). This should be consistent in all dialogs.

Also the button style is different in different dialogs.
Some dialogs shows buttons in square brackets, for example [Ok] [ Reset ] [ Cancel ].

The other dialogs shows the buttons without square brackets, for example:

 Delete Cancel

This should also be consistent. I suggest to use button style without square brackets.
TagsNo tags attached.
Build1519

Activities

samlyukov

2008-11-30 11:57

reporter   bugnote:0002075

Last edited: 2008-12-19 12:37

ну, если сделать навроде этого:
Index: message.cpp
===================================================================
--- message.cpp (revision 2313)
+++ message.cpp (working copy)
@@ -168,7 +168,7 @@
 
   // предварительный обсчет максимального размера.
   for (BtnLength=0,I=0;I<static_cast<DWORD>(Buttons);I++) //??
- BtnLength+=HiStrlen(Items[I+StrCount])+2;
+ BtnLength+=HiStrlen(Items[I+StrCount])+2+4;
 
   for (MaxLength=BtnLength,I=0;I<StrCount;I++)
   {
@@ -329,7 +329,7 @@
         PtrMsgDlg->Focus=1;
         PtrMsgDlg->DefaultButton=1;
         TypeItem=DI_BUTTON;
- FlagsItem=DIF_CENTERGROUP|DIF_NOBRACKETS;
+ FlagsItem=DIF_CENTERGROUP/*|DIF_NOBRACKETS*/;
         IsButton=TRUE;
         FirstButtonIndex=CurItem+1;
         LastButtonIndex=CurItem;
@@ -341,7 +341,7 @@
       if(IsButton)
       {
         PtrMsgDlg->Y1=Y2-Y1-2;
- PtrMsgDlg->strData = string(L" ")+string(CPtrStr)+string(L" "); //BUGBUG!!!
+ PtrMsgDlg->strData = /*string(L" ")+*/string(CPtrStr)/*+string(L" ")*/; //BUGBUG!!!
         LastButtonIndex++;
       }
       else
-----------------------------------------------------------
получим нечто похожее-однородное, только надо ли...

samlyukov

2008-11-30 12:08

reporter   bugnote:0002076

хотя, я тут некоторое время поюзал - мне понравилось :-)

alexmitin

2008-11-30 21:34

reporter   bugnote:0002079

А можно попробовать такой вариант?

1) Убрать square brackets из кнопок вообще.
2) Добавить по два пробела с каждой стороны к названию кнопки. Для кнопки OK использовать 4 пробела с каждой стороны.
3) Использовать один пробел МЕЖДУ кнопками.

-------------------
По поводу пункта 1: границы кнопок хорошо задаются фоновым цветом кнопок (Options, Colors, Dialog, Buttons/Selected Buttons). Square brackets только создают дополнительную помеху при чтении.

Пункт 2: Можно расширить таким образом.
1) Для кнопок, у которых название 2 символа добавлять 4 пробела с каждой стороны.
2) Для кнопок, у которых название 3-4 символа добавлять 3 пробела с каждой стороны.
3) Для остальных кнопок два пробела с каждой стороны.

Смотри аттачмент с примерами. Открывать файл из аттачмента командой:

hlf:Mantis647.hlf

2008-11-30 21:34

 

Mantis647.hlf (9,205 bytes)

alexmitin

2008-12-01 03:22

reporter   bugnote:0002080

Файл Mantis647.2.hlf содежит примеры использования нового стиля кнопок в различных диалогах.

Обратите внимание, что кнопки внизу диалога отделены от верхней части сплошной линией. Это также облегчает чтение диалога.

2008-12-01 03:23

 

Mantis647.2.hlf (30,220 bytes)

samlyukov

2008-12-01 13:27

reporter   bugnote:0002091

>Пункт 2: Можно расширить таким образом.
Есть мнение, что это лучше делать не программными средствами, а непосредственно в lng-файле.

alexmitin

2010-04-26 14:25

reporter   bugnote:0005275

Пожалуйста сделайте настраиваемым вид кнопок:

  1) Вариант 1: c квадратными скобками
  2) Вариант 2: без квадратных скобок. Смотри комментарий http://bugs.farmanager.com/view.php?id=647#c2079

Можно через регистр. Хотя, если делать настройку через UI, то добавить в "Dialog Settings":

English: [x] Use square brackets in button style.
Русский: [x] Использовать скобки в стиле кнопок.

-------------

По поводу bug #315:
К сожалению я не могу добавить комментарий к закрытому issue (не моему). Поэтому добавлю здесь. Тем более, что оба issue связаны между собой.

В 315 был запрос на настройку цвета для кнопок по умолчанию, а не к изменению вида скобок. В FAR 2.0.1522

Issue History

Date Modified Username Field Change
2008-11-30 04:41 alexmitin New Issue
2008-11-30 04:41 alexmitin Status new => assigned
2008-11-30 04:41 alexmitin Assigned To => vskirdin
2008-11-30 11:57 samlyukov Note Added: 0002075
2008-11-30 12:00 samlyukov Note Edited: 0002075
2008-11-30 12:08 samlyukov Note Added: 0002076
2008-11-30 21:34 alexmitin Note Added: 0002079
2008-11-30 21:34 alexmitin File Added: Mantis647.hlf
2008-12-01 03:22 alexmitin Note Added: 0002080
2008-12-01 03:23 alexmitin File Added: Mantis647.2.hlf
2008-12-01 13:27 samlyukov Note Added: 0002091
2008-12-19 12:37 samlyukov Note Edited: 0002075
2010-04-25 16:59 DrKnS Build => 1519
2010-04-25 16:59 DrKnS Status assigned => closed
2010-04-25 16:59 DrKnS Assigned To vskirdin => DrKnS
2010-04-25 16:59 DrKnS Resolution open => fixed
2010-04-25 16:59 DrKnS Fixed in Version => 2.0
2010-04-26 14:25 alexmitin Note Added: 0005275
2010-04-26 14:25 alexmitin Status closed => feedback
2010-04-26 14:25 alexmitin Resolution fixed => reopened
2011-12-23 08:50 alexy Project Far Manager => Wishes
2012-12-14 16:11 samlyukov Status feedback => resolved
2013-02-03 00:33 alexy Severity tweak => feature
2013-02-04 00:30 alexy Status resolved => closed
2013-02-04 00:30 alexy Resolution reopened => fixed