Board.KolibriOS.org

Второй раз скачал я Ещё не тестил.

_________________
Звиздеть не мешки ворочать

Резюмирую:
Расстояние от рамки до текста увеличу.
Чтобы не единолично все решать, надо обсудить два не до конца ясных для меня вопроса: приделывать ли эдитбокс и оставить ли добавленное мной ранее спорное автозакрытие при успешном сохранении по умолчанию?


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

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

Эмоции, по смыслу перегиба нет. Устроил опрос для ясности.

_________________
Звиздеть не мешки ворочать

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

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

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

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

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

Протестировал последнюю версию, как обещал. В целом, программа понравилась. (Да, я написал это.)
Замечания/пожелания/найденные баги:

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, сам по себе.
Заливать или нет на SVN у нас решает Leency, так что я не буду "брать на себя слишком много". Как он решит, так и делай.

Использование OpenDialog для выбора пути сохранения на флопик вводит пользователя в заблуждение. Формально в этом случае на флопик и рамдиск должна быть попытка записи файла kolibri.img, а не образа. В этой же версии программы эта логика подменяется функцией записи образа.

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

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

Во! Спасибо. Всегда полезно получить мнение людей со свежим взглядом. Многие нелогичности мне бы и в голову не пришли.Теперь по пунктам: Принято - исправлю.
2 человека против автозакрытия по умолчанию, остальным пофиг - значит, сделаю.
За этот пункт спасибо отдельное - сам бы не скоро дошел до этого - исправлю.
Тут соглашусь с Mario - показывать не все диски - имхо нелогичное поведение, и диски определяет opendialog сам - rdsave тут действительно не при чем. И, думаю, можно смириться с тем, что при первом сохранении пользователь увидит все диски, которые есть в системе и выберет нужный ему. Дальше же opendialog почти не будет использоваться. Насчет некоторого разрыва шаблона с сохранением на флопик - согласен. Использовал первый пришедший в голову хак. Надо будет еще потом покумекать и сделать таки по человечески.
Мнение принято, только что насчет невмещающихся путей? Не критично?
Тут не согласен: если откроется два окна рядом, на какое окно должен обращать внимание пользователь и в каком из них делать манипуляции? Разве что можно сделать, чтобы бесполезное в данном случае окно rdsave было перекрыто, но немного видно из под окна opendialog. Подумаю над этим.
Опять за свое?

Исправил.

Автозакрытие по умолчанию оключил.

Исправил.

Насчет некоторого разрыва шаблона с сохранением на флопик - согласен. Использовал первый пришедший в голову хак. Надо будет еще потом покумекать и сделать таки по человечески - пока без изменений.

Мнение принято, только что насчет невмещающихся путей? Не критично? - пока без изменений.

Тут не согласен: если откроется два окна рядом, на какое окно должен обращать внимание пользователь и в каком из них делать манипуляции? Разве что можно сделать, чтобы бесполезное в данном случае окно rdsave было перекрыто, но немного видно из под окна opendialog. Подумаю над этим. - Без изменений - opendialog судя по всему так написан, что его окно рисуется от центра вызывающего приложения.

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

Leency уже давно решил, а теперь и я тоже. Программа работает стабильно, и по сравнению с 1.3 есть множество полюшек. (Спасибо eAndrew за идеи). Потому r. 4213.

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

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

* /rd: Если это не было понятно из моего предыдущего сообщения, то я смог сохранить рамдиск сам на себя. Вернее, сохранил первые 120 килобайт файла kolibri.img, потом на рамдиске просто кончилось место. Если потом этот же рамдиск сохранить на настоящий диск, то он так и сохранится, с 0 байт свободных и файлом kolibri.img внутри файла kolibri.img. Я думаю, что такую возможность давать пользователю опасно, так как с 0 байт свободных некоторые программы будут работать некорректно, и можно получить неверное представление, что /rd - это не рамдиск, а что-то другое.
* /fdX: Вы же сами согласились с тем, что способ сохранения вводит пользователя в заблуждение. Пользователь думает, что сохраняет на флопик файл kolibri.img, а на самом деле, сохраняется образ. Не зря же в предыдущей версии RDSAVE для сохранения на флопик были отдельные кнопки. Почему бы их не сделать и в этой версии?

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

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

А разве я говорил, что при сохранении на флопик, путь в ini прописывать не нужно? В ini ведём себя, как и сейчас. Только путь пишем без "kolibri.img" в случае флопика. Т.е. END ведёт себя без изменений: есть rdsave.ini - сохраняет туда, куда указывает путь; нет rdsave.ini - открывает RDSAVE.
Открывать окно RDSAVE вместо окна OpenDialog. Либо продолжать открывать OpenDialog, пользователь нажимает Cancel, и потом нажимает на кнопку "Сохранить на floppy1". Так пользователь сразу понимает, что сохранение на флопик отличается от сохранения на другие диски.
Без отдельной кнопки для "my path", твоя версия не становится хуже существующей, поэтому я вполне могу без неё обойтись (и обещаю больше не просить). Точно так же, как открытие OpenDialog поверх RDSAVE - если ты не заметил, то после твоего возражения, я больше ни слова не сказал, так как это не баг. Но поведение с сохранением на рамдиск и флопик - это уже явные баги и нелогичности, причём сохранение на флопик - ухудшение по сравнению с предыдущей версией.