Board.KolibriOS.org

Official KolibriOS board
It is currently Mon Aug 19, 2019 11:47 pm

All times are UTC+03:00




Post new topic  Reply to topic  [ 157 posts ]  Go to page Previous 16 7 8 9 10 11 Next
Author Message
PostPosted: Thu Nov 07, 2013 7:40 pm 
Offline
Designer
User avatar

Joined: Thu Jan 25, 2007 3:33 pm
Posts: 5054
Второй раз скачал я :) Ещё не тестил.

_________________
Через тернии к звездам


Last edited by Leency on Fri Nov 08, 2013 12:29 pm, edited 2 times in total.

Top
   
PostPosted: Thu Nov 07, 2013 7:47 pm 
Offline
User avatar

Joined: Fri Jan 27, 2006 3:06 pm
Posts: 1071
Резюмирую:
Расстояние от рамки до текста увеличу.
Чтобы не единолично все решать, надо обсудить два не до конца ясных для меня вопроса: приделывать ли эдитбокс и оставить ли добавленное мной ранее спорное автозакрытие при успешном сохранении по умолчанию?


PS: Leency, перегибать палку тоже не стоит.

PPS: И да, чуть не забыл третий главный вопрос: заливать ли?


Last edited by Heavyiron on Thu Nov 07, 2013 8:54 pm, edited 2 times in total.

Top
   
PostPosted: Thu Nov 07, 2013 7:49 pm 
Offline
Designer
User avatar

Joined: Thu Jan 25, 2007 3:33 pm
Posts: 5054
Эмоции, по смыслу перегиба нет. Устроил опрос для ясности.

_________________
Через тернии к звездам


Top
   
PostPosted: Thu Nov 07, 2013 8:39 pm 
Offline
Kernel Developer

Joined: Sun Feb 10, 2013 12:37 pm
Posts: 2329
Heavyiron wrote:
убирать ли автозакрытие при успешном сохранении по умолчанию?

Вот как ловко вопрос сформулирован. Посторонний наблюдатель сразу подумает "Так это уже оно 9 лет наверное так было, тогда чего менять? Оставляем разумеется!" :lol:

_________________
Всем чмоки в этом проекте! Засуньте эти 11 лет себе в жопу!


Top
   
PostPosted: Thu Nov 07, 2013 8:42 pm 
Offline
User avatar

Joined: Sat Oct 05, 2013 9:32 pm
Posts: 386
Есть такая идея (конечно это урезание функционала, но что поделать):
Сама программа RDSave ничего сохранять не будет(точнее не будет считаться главной программой для сохранения). Вместо этого нужно добавить чекбокс в END "Сохранить образ?", который будет сам фоново сохранять образ перед выходом, и кнопку настроек, которая будет открывать RDSave для настроек сохранения образа.
В RDsave можно оставить кнопку сохранения, но сделать ее не основной, а так.. дополнительной, а на ее месте поставить кнопку "OK", который возвращает фокус обратно на END.
Вот как.


Last edited by e-andrew on Thu Nov 07, 2013 8:44 pm, edited 1 time in total.

Top
   
PostPosted: Thu Nov 07, 2013 8:43 pm 
Offline
User avatar

Joined: Fri Jan 27, 2006 3:06 pm
Posts: 1071
Mario_r4 wrote:
Heavyiron wrote:
убирать ли автозакрытие при успешном сохранении по умолчанию?

Вот как ловко вопрос сформулирован. Посторонний наблюдатель сразу подумает "Так это уже оно 9 лет наверное так было, тогда чего менять? Оставляем разумеется!" :lol:

Вопрос подкорректировал - без злого умысла - просто неумение точно выразить мысль.


Top
   
PostPosted: Thu Nov 07, 2013 8:50 pm 
Offline
User avatar

Joined: Fri Jan 27, 2006 3:06 pm
Posts: 1071
eAndrew wrote:
Есть такая идея (конечно это урезание функционала, но что поделать):
Сама программа RDSave ничего сохранять не будет(точнее не будет считаться главной программой для сохранения). Вместо этого нужно добавить чекбокс в END "Сохранить образ?", который будет сам фоново сохранять образ перед выходом, и кнопку настроек, которая будет открывать RDSave для настроек сохранения образа.
В RDsave можно оставить кнопку сохранения, но сделать ее не основной, а так.. дополнительной, а на ее месте поставить кнопку "OK", который возвращает фокус обратно на END.
Вот как.

Немного не то, но планируется. Перенос функционала из rdsave в end считаю нецелесообразным, но в End вместо кнопки "Сохранить настройки" я сделаю чекбокс (как ты и предлагал), который будет запускать rdsave с параметром "h" - сохранение без отрисовки окна rdsave в папку, указанную в rdsave.ini. Если по указанному пути сохранить не удается - откроется opendialog. End будет ждать закрытия rdsave и дальше перезагружать или выключать комп. Почва в текущей версии rdsave для этого уже подготовлена.


Top
   
PostPosted: Fri Nov 08, 2013 5:44 am 
Offline
Public Relations
User avatar

Joined: Mon Jun 07, 2010 12:01 pm
Posts: 1879
Heavyiron wrote:
Чтобы не единолично все решать, надо обсудить два не до конца ясных для меня вопроса: приделывать ли эдитбокс и оставить ли добавленное мной ранее спорное автозакрытие при успешном сохранении по умолчанию?
Протестировал последнюю версию, как обещал. В целом, программа понравилась. (Да, я написал это.)
Замечания/пожелания/найденные баги:

1) Если файл rdsave.ini не существовал, то при создании файла самой программой, параметр "autoclose=1" не добавляется (добавляется только параметр path). ИМХО, это не совсем верное поведение при создании INI файлов (если файл генерируется автоматически, то все параметры по умолчанию должны присутствовать).
2) Хотелось бы, чтобы автозакрытие при успешном сохранении по умолчанию в автосборке было отключено. Раз этот параметр в INI, то любой может его себе установить включенным.
3) Race condition с файлом настроек - сначала сохраняется образ со старым файлом rdsave.ini по новому пути, потом путь в файле изменяется на новый, и получается, что в образе файл ещё старый. Чтобы в образе тоже оказался новый файл, приходится сохранять 2 раза подряд в одно и то же место.
4) Я не знаю, функция ли это RDSAVE или OpenDialog, но ИМХО в списке дисков OpenDialog не должны присутствовать флопики /fdX и рамдиск /rd:
  • /rd: Сохранять рамдиск сам на себя - то ещё извращение, к тому же эта затея заранее обречена на провал, только оставшиеся 100 килобайт закончатся.
  • /fdX: На флопик сохраняется сам диск, а OpenDialog создаёт ощущение, что сохраняет файл kolibri.img - получается разрыв шаблона. ИМХО сохранение на флопик должно осуществляться отдельной кнопкой (как в существующей версии 1.3), и не выводиться через OpenDialog
5) ИМХО EditBox таки не помешал бы.
6) Если файл rdsave.ini не существовал, то при запуске программы, окно OpenDialog выводится поверх окна программы. Выводить сразу окно OpenDialog, возможно, и неплохая идея, но лучше тогда выводить оба окна в разных местах экрана, чтобы одно не заслоняло другое. Так будет понятно, что это запустился RDSAVE, который запустил OpenDialog. А сейчас кажется, что запустился только OpenDialog, сам по себе.
Heavyiron wrote:
PPS: И да, чуть не забыл третий главный вопрос: заливать ли?
Заливать или нет на SVN у нас решает Leency, так что я не буду "брать на себя слишком много". Как он решит, так и делай.


Top
   
PostPosted: Fri Nov 08, 2013 7:39 am 
Offline
Kernel Developer

Joined: Sun Feb 10, 2013 12:37 pm
Posts: 2329
Использование OpenDialog для выбора пути сохранения на флопик вводит пользователя в заблуждение. Формально в этом случае на флопик и рамдиск должна быть попытка записи файла kolibri.img, а не образа. В этой же версии программы эта логика подменяется функцией записи образа.

То что OpenDialog показывает полный список доступных дисков - это нормальная ситуация и реакция. Было бы очень странно, если бы OpenDialog не показывал все доступные диски.

_________________
Всем чмоки в этом проекте! Засуньте эти 11 лет себе в жопу!


Top
   
PostPosted: Fri Nov 08, 2013 12:21 pm 
Offline
User avatar

Joined: Fri Jan 27, 2006 3:06 pm
Posts: 1071
Во! Спасибо. Всегда полезно получить мнение людей со свежим взглядом. Многие нелогичности мне бы и в голову не пришли.Теперь по пунктам:
yogev_ezra wrote:
Замечания/пожелания/найденные баги:
1) Если файл rdsave.ini не существовал, то при создании файла самой программой, параметр "autoclose=1" не добавляется (добавляется только параметр path). ИМХО, это не совсем верное поведение при создании INI файлов (если файл генерируется автоматически, то все параметры по умолчанию должны присутствовать).
Принято - исправлю.
yogev_ezra wrote:
2) Хотелось бы, чтобы автозакрытие при успешном сохранении по умолчанию в автосборке было отключено. Раз этот параметр в INI, то любой может его себе установить включенным.
2 человека против автозакрытия по умолчанию, остальным пофиг - значит, сделаю.
yogev_ezra wrote:
3) Race condition с файлом настроек - сначала сохраняется образ со старым файлом rdsave.ini по новому пути, потом путь в файле изменяется на новый, и получается, что в образе файл ещё старый. Чтобы в образе тоже оказался новый файл, приходится сохранять 2 раза подряд в одно и то же место.
За этот пункт спасибо отдельное - сам бы не скоро дошел до этого - исправлю.
yogev_ezra wrote:
4) Я не знаю, функция ли это RDSAVE или OpenDialog, но ИМХО в списке дисков OpenDialog не должны присутствовать флопики /fdX и рамдиск /rd:
  • /rd: Сохранять рамдиск сам на себя - то ещё извращение, к тому же эта затея заранее обречена на провал, только оставшиеся 100 килобайт закончатся.
  • /fdX: На флопик сохраняется сам диск, а OpenDialog создаёт ощущение, что сохраняет файл kolibri.img - получается разрыв шаблона. ИМХО сохранение на флопик должно осуществляться отдельной кнопкой (как в существующей версии 1.3), и не выводиться через OpenDialog
Mario_r4 wrote:
Использование OpenDialog для выбора пути сохранения на флопик вводит пользователя в заблуждение. Формально в этом случае на флопик и рамдиск должна быть попытка записи файла kolibri.img, а не образа. В этой же версии программы эта логика подменяется функцией записи образа.
То что OpenDialog показывает полный список доступных дисков - это нормальная ситуация и реакция. Было бы очень странно, если бы OpenDialog не показывал все доступные диски.
Тут соглашусь с Mario - показывать не все диски - имхо нелогичное поведение, и диски определяет opendialog сам - rdsave тут действительно не при чем. И, думаю, можно смириться с тем, что при первом сохранении пользователь увидит все диски, которые есть в системе и выберет нужный ему. Дальше же opendialog почти не будет использоваться. Насчет некоторого разрыва шаблона с сохранением на флопик - согласен. Использовал первый пришедший в голову хак. Надо будет еще потом покумекать и сделать таки по человечески.
yogev_ezra wrote:
5) ИМХО EditBox таки не помешал бы.
Мнение принято, только что насчет невмещающихся путей? Не критично?
yogev_ezra wrote:
6) Если файл rdsave.ini не существовал, то при запуске программы, окно OpenDialog выводится поверх окна программы. Выводить сразу окно OpenDialog, возможно, и неплохая идея, но лучше тогда выводить оба окна в разных местах экрана, чтобы одно не заслоняло другое. Так будет понятно, что это запустился RDSAVE, который запустил OpenDialog. А сейчас кажется, что запустился только OpenDialog, сам по себе.
Тут не согласен: если откроется два окна рядом, на какое окно должен обращать внимание пользователь и в каком из них делать манипуляции? Разве что можно сделать, чтобы бесполезное в данном случае окно rdsave было перекрыто, но немного видно из под окна opendialog. Подумаю над этим.
yogev_ezra wrote:
Заливать или нет на SVN у нас решает Leency, так что я не буду "брать на себя слишком много". Как он решит, так и делай.
Опять за свое?


Top
   
PostPosted: Tue Nov 12, 2013 1:49 am 
Offline
User avatar

Joined: Fri Jan 27, 2006 3:06 pm
Posts: 1071
yogev_ezra wrote:
1) Если файл rdsave.ini не существовал, то при создании файла самой программой, параметр "autoclose=1" не добавляется (добавляется только параметр path). ИМХО, это не совсем верное поведение при создании INI файлов (если файл генерируется автоматически, то все параметры по умолчанию должны присутствовать).
Исправил.

yogev_ezra wrote:
2) Хотелось бы, чтобы автозакрытие при успешном сохранении по умолчанию в автосборке было отключено. Раз этот параметр в INI, то любой может его себе установить включенным.
Автозакрытие по умолчанию оключил.

yogev_ezra wrote:
3) Race condition с файлом настроек - сначала сохраняется образ со старым файлом rdsave.ini по новому пути, потом путь в файле изменяется на новый, и получается, что в образе файл ещё старый. Чтобы в образе тоже оказался новый файл, приходится сохранять 2 раза подряд в одно и то же место.
Исправил.

yogev_ezra wrote:
4) Я не знаю, функция ли это RDSAVE или OpenDialog, но ИМХО в списке дисков OpenDialog не должны присутствовать флопики /fdX и рамдиск /rd:
  • /rd: Сохранять рамдиск сам на себя - то ещё извращение, к тому же эта затея заранее обречена на провал, только оставшиеся 100 килобайт закончатся.
  • /fdX: На флопик сохраняется сам диск, а OpenDialog создаёт ощущение, что сохраняет файл kolibri.img - получается разрыв шаблона. ИМХО сохранение на флопик должно осуществляться отдельной кнопкой (как в существующей версии 1.3), и не выводиться через OpenDialog
Mario_r4 wrote:
Использование OpenDialog для выбора пути сохранения на флопик вводит пользователя в заблуждение. Формально в этом случае на флопик и рамдиск должна быть попытка записи файла kolibri.img, а не образа. В этой же версии программы эта логика подменяется функцией записи образа.
То что OpenDialog показывает полный список доступных дисков - это нормальная ситуация и реакция. Было бы очень странно, если бы OpenDialog не показывал все доступные диски.
Насчет некоторого разрыва шаблона с сохранением на флопик - согласен. Использовал первый пришедший в голову хак. Надо будет еще потом покумекать и сделать таки по человечески - пока без изменений.

yogev_ezra wrote:
5) ИМХО EditBox таки не помешал бы.
Мнение принято, только что насчет невмещающихся путей? Не критично? - пока без изменений.

yogev_ezra wrote:
6) Если файл rdsave.ini не существовал, то при запуске программы, окно OpenDialog выводится поверх окна программы. Выводить сразу окно OpenDialog, возможно, и неплохая идея, но лучше тогда выводить оба окна в разных местах экрана, чтобы одно не заслоняло другое. Так будет понятно, что это запустился RDSAVE, который запустил OpenDialog. А сейчас кажется, что запустился только OpenDialog, сам по себе.
Тут не согласен: если откроется два окна рядом, на какое окно должен обращать внимание пользователь и в каком из них делать манипуляции? Разве что можно сделать, чтобы бесполезное в данном случае окно rdsave было перекрыто, но немного видно из под окна opendialog. Подумаю над этим. - Без изменений - opendialog судя по всему так написан, что его окно рисуется от центра вызывающего приложения.

Кроме того, теперь при успешном сохранении @notify пишет куда сохранился образ.

yogev_ezra wrote:
Заливать или нет на SVN у нас решает Leency, так что я не буду "брать на себя слишком много". Как он решит, так и делай.
Leency уже давно решил, а теперь и я тоже. Программа работает стабильно, и по сравнению с 1.3 есть множество полюшек. (Спасибо eAndrew за идеи). Потому r. 4213.


Top
   
PostPosted: Tue Nov 12, 2013 2:33 am 
Offline
Kernel Developer

Joined: Sun Feb 10, 2013 12:37 pm
Posts: 2329
Heavyiron wrote:
opendialog судя по всему так написан, что его окно рисуется от центра вызывающего приложения.

Да, так и есть. Это сделано умышленно. Если окно приложения находится в одном углу, а OpenDialog откроется в противоположном, то буден нелогично с точки зрения конечного пользователя.

_________________
Всем чмоки в этом проекте! Засуньте эти 11 лет себе в жопу!


Top
   
PostPosted: Fri Nov 15, 2013 1:22 am 
Offline
Public Relations
User avatar

Joined: Mon Jun 07, 2010 12:01 pm
Posts: 1879
Heavyiron wrote:
yogev_ezra wrote:
4) Я не знаю, функция ли это RDSAVE или OpenDialog, но ИМХО в списке дисков OpenDialog не должны присутствовать флопики /fdX и рамдиск /rd:
  • /rd: Сохранять рамдиск сам на себя - то ещё извращение, к тому же эта затея заранее обречена на провал, только оставшиеся 100 килобайт закончатся.
  • /fdX: На флопик сохраняется сам диск, а OpenDialog создаёт ощущение, что сохраняет файл kolibri.img - получается разрыв шаблона. ИМХО сохранение на флопик должно осуществляться отдельной кнопкой (как в существующей версии 1.3), и не выводиться через OpenDialog
Mario_r4 wrote:
Использование OpenDialog для выбора пути сохранения на флопик вводит пользователя в заблуждение. Формально в этом случае на флопик и рамдиск должна быть попытка записи файла kolibri.img, а не образа. В этой же версии программы эта логика подменяется функцией записи образа.
То что OpenDialog показывает полный список доступных дисков - это нормальная ситуация и реакция. Было бы очень странно, если бы OpenDialog не показывал все доступные диски.
Тут соглашусь с Mario - показывать не все диски - имхо нелогичное поведение, и диски определяет opendialog сам - rdsave тут действительно не при чем. И, думаю, можно смириться с тем, что при первом сохранении пользователь увидит все диски, которые есть в системе и выберет нужный ему. Дальше же opendialog почти не будет использоваться. Насчет некоторого разрыва шаблона с сохранением на флопик - согласен. Использовал первый пришедший в голову хак. Надо будет еще потом покумекать и сделать таки по человечески.
* /rd: Если это не было понятно из моего предыдущего сообщения, то я смог сохранить рамдиск сам на себя. Вернее, сохранил первые 120 килобайт файла kolibri.img, потом на рамдиске просто кончилось место. Если потом этот же рамдиск сохранить на настоящий диск, то он так и сохранится, с 0 байт свободных и файлом kolibri.img внутри файла kolibri.img. Я думаю, что такую возможность давать пользователю опасно, так как с 0 байт свободных некоторые программы будут работать некорректно, и можно получить неверное представление, что /rd - это не рамдиск, а что-то другое.
* /fdX: Вы же сами согласились с тем, что способ сохранения вводит пользователя в заблуждение. Пользователь думает, что сохраняет на флопик файл kolibri.img, а на самом деле, сохраняется образ. Не зря же в предыдущей версии RDSAVE для сохранения на флопик были отдельные кнопки. Почему бы их не сделать и в этой версии?

Я прошу вас двоих скооперироваться, и сделать в OpenDialog поддержку вызова с параметром "ramdisk". При вызове с этим параметром, не показываются диски /rd, /fd1, /fd2. RDSAVE вызывает OpenDialog с этим параметром, и пользователь не вводится в заблуждение (и не совершает потенциально опасных действий). А для сохранения на флопик вполне можно вернуть отдельные кнопки (без упоминания kolibri.img, как в старой версии), так будет выглядеть логично.


Top
   
PostPosted: Fri Nov 15, 2013 2:42 am 
Offline
User avatar

Joined: Fri Jan 27, 2006 3:06 pm
Posts: 1071
Над флопиками продолжаю думать. В текущей версии есть по сути 2 сущности: 1) путь по умолчанию, который при первом сохранении превращается в 2) путь последнего успешного сохранения на данной системе который и используется в дальнейшем по умолчанию - программа старается быть незаметной. Если делать отдельную кнопку (или чекбокс?) для флопика, то сразу появляется много нестыковок. Куда, например, будет сохранять образ программа End по умолчанию: на флопик или по заданному в ini пути на /hd или /usb при наличии в системе всех этих вариантов? Или, например, в системе есть флопик, а дефолтный путь из ini не найден, что тогда рисовать в поле пути? Теперь открывается opendialog для выбора пути, но если будет отдельная кнопка "/fd/1", то такое поведение уже нелогично - ведь может на флопик и нужно сохранить. Да и не вернемся ли мы к разговору о том, что раз есть отдельная кнопка для флопика, то почему бы не добавить еще и кнопку для "/my favorit path"? В итоге программа опять по сути превратится в менее удобную на мой взгляд версию 1.3, но с косметическими изменениями. Да, есть нелогичности в версии 1.4, но ИМХО в них несложно разобраться и понять, что: 1) не нужно сохранять образ в самого себя (это, кстати, несложно заблокировать уже после выбора в opendialog-е /rd/1/), 2) пока используется хак для сохранения на флопик через opendialog, котороый в билжайшем времени постараюсь заменить на более удачное решение (только нужно некоторое время на размышления).


Top
   
PostPosted: Fri Nov 15, 2013 3:59 am 
Offline
Public Relations
User avatar

Joined: Mon Jun 07, 2010 12:01 pm
Posts: 1879
Heavyiron wrote:
Если делать отдельную кнопку (или чекбокс?) для флопика, то сразу появляется много нестыковок. Куда, например, будет сохранять образ программа End по умолчанию: на флопик или по заданному в ini пути на /hd или /usb при наличии в системе всех этих вариантов?
А разве я говорил, что при сохранении на флопик, путь в ini прописывать не нужно? В ini ведём себя, как и сейчас. Только путь пишем без "kolibri.img" в случае флопика. Т.е. END ведёт себя без изменений: есть rdsave.ini - сохраняет туда, куда указывает путь; нет rdsave.ini - открывает RDSAVE.
Heavyiron wrote:
Или, например, в системе есть флопик, а дефолтный путь из ini не найден, что тогда рисовать в поле пути? Теперь открывается opendialog для выбора пути, но если будет отдельная кнопка "/fd/1", то такое поведение уже нелогично - ведь может на флопик и нужно сохранить.
Открывать окно RDSAVE вместо окна OpenDialog. Либо продолжать открывать OpenDialog, пользователь нажимает Cancel, и потом нажимает на кнопку "Сохранить на floppy1". Так пользователь сразу понимает, что сохранение на флопик отличается от сохранения на другие диски.
Heavyiron wrote:
Да и не вернемся ли мы к разговору о том, что раз есть отдельная кнопка для флопика, то почему бы не добавить еще и кнопку для "/my favorit path"? В итоге программа опять по сути превратится в менее удобную на мой взгляд версию 1.3, но с косметическими изменениями. Да, есть нелогичности в версии 1.4, но ИМХО в них несложно разобраться и понять, что: 1) не нужно сохранять образ в самого себя (это, кстати, несложно заблокировать уже после выбора в opendialog-е /rd/1/), 2) пока используется хак для сохранения на флопик через opendialog, котороый в билжайшем времени постараюсь заменить на более удачное решение (только нужно некоторое время на размышления).
Без отдельной кнопки для "my path", твоя версия не становится хуже существующей, поэтому я вполне могу без неё обойтись (и обещаю больше не просить). Точно так же, как открытие OpenDialog поверх RDSAVE - если ты не заметил, то после твоего возражения, я больше ни слова не сказал, так как это не баг. Но поведение с сохранением на рамдиск и флопик - это уже явные баги и нелогичности, причём сохранение на флопик - ухудшение по сравнению с предыдущей версией.


Top
   
Display posts from previous:  Sort by  
Post new topic  Reply to topic  [ 157 posts ]  Go to page Previous 16 7 8 9 10 11 Next

All times are UTC+03:00


Who is online

Users browsing this forum: No registered users and 3 guests


You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot post attachments in this forum

Search for:
Jump to:  
cron
Powered by phpBB® Forum Software © phpBB Limited