Opened 8 years ago

Closed 8 years ago

Last modified 6 years ago

#234 closed баг (fixed)

Заменен заголовок окна аутентификации, и он не работает

Reported by: alx Owned by: dimag
Priority: minor Milestone: 2 очередь
Component: ПО MC04-Dispatcher. Пульт диспетчера/техника Keywords: wm header
Cc: san

Description

Заметил, что в r253 вместо штатного хедера окна аутентификации, который рисует Window manager, стал появляться какой-то другой.

Мало того, что его стиль отличается от стиля заголовков всех прочих окон, так он еще и не работает - как минимум, не позволяет перемещать окно мышью за заголовок, нет кнопки вызова меню, которую рисовал мой Window Manager и т.п.

Я считаю, надо вернуть окну заголовок и все сопутствующие его функции, за которые отвечает Window manager.

Change History (17)

comment:1 by alx, 8 years ago

То же самое наблюдается у окна с информацией о пользователе.

comment:2 by dimag, 8 years ago

Алексей твои замечания? По макету интерфейса эти функции не нужны.

in reply to:  description ; comment:3 by san, 8 years ago

Replying to alx:

Заметил, что в r253 вместо штатного хедера окна аутентификации, который рисует Window manager, стал появляться какой-то другой.
Мало того, что его стиль отличается от стиля заголовков всех прочих окон

Дима переделал вид окна в соответствие с макетом от дизайнера #211

так он еще и не работает

А вот это плохо

не позволяет перемещать окно мышью за заголовок

Думаю это надо считать багом

Я считаю, надо вернуть окну заголовок и все сопутствующие его функции, за которые отвечает Window manager.

Я не против, но окно и заголовок должно соответствовать по внешнему виду макета #211 (хотя и тут есть ошибка - кнопки развернуть не должно быть у окна авторизации)

Last edited 8 years ago by san (previous) (diff)

in reply to:  3 ; comment:4 by alx, 8 years ago

Replying to san:

Дима переделал вид окна в соответствие с макетом от дизайнера #211

:) Там на картинке дизайнера еще присутствует, например, голубой фон вокруг окна. Почему его не реализовали? :) :) Тогда ведь по логике надо было сделать и этот фон тоже...

Мне кажется, переделывая вид окна в соответствии с предложенным дизайном, надо думать, а не просто механически его перерисовывать. А то ведь можно дойти до абсурда, просто взяв картинку дизайнера целиком и поместив ее в окно программы... :)

По-моему очевидно, что элемент с тремя кнопками и надписью "TelecomApp - Authorization" - это и есть заголовок окна программы. И нарисован он, как и голубой фон вокруг, просто для лучшего понимания, что тут вообще изображено. А собственно же дизайн, который необходимо применить - это все что ниже заголовка, внутри окна...

Кроме того, в #211 я не вижу указаний о том, что такие функции как перемещение окна по экрану или перемещения окна с одного десктопа на другой не нужны.

Я считаю, надо вернуть окну заголовок и все сопутствующие его функции, за которые отвечает Window manager.

Я не против, но окно и заголовок должно соответствовать по внешнему виду макета #211

Я думаю, это пользователю решать, как должны выглядеть заголовки окон на его дисплее. Если у пользователя настроено, к примеру, что заголовки всех окон имеют белый текст на темно-сером фоне, а кнопка закрытия окна находится в левом углу - значит ему так удобнее, привычнее и больше нравится. И я не понимаю, с какой стати одно конкретное окно одной конкретной программы вдруг должно иметь заголовок с кнопкой в правом углу и с черным текстом на белом фоне. Чтобы ему труднее было эту кнопку найти? :)

Есть вещи, за которые отвечает наша программа (это содержимое ее окон), а есть вещи, за которые отвечают другие программы, например window mnager (оформление окон, их расположение, перемещение и т.п.). И мне кажется, не надо залезать в чужую зону ответственности. Оставьте windows management Window Manager'у...

in reply to:  4 comment:5 by san, 8 years ago

Алексей, логически я с тобой согласен, но ...

Я думаю, это пользователю решать, как должны выглядеть заголовки окон на его дисплее.

Я не уверен что это так. Заказчик хочет чтобы вид программы был "всегда" одинаковый и хеадер окна в том числе.
Я уточню отдельно этот момент.

comment:6 by alx, 8 years ago

Здесь я вынужден дать немного отвлеченный комментарий. Ты здесь часто пишешь "Заказчик хочет...". Однако, в моем понимании, "заказчик" - это тот, кто заказал выполнение работы и платит за это деньги. Соответственно, исполнитель, получая деньги, берет на себя обязательство выполнить работу в соответствии с ТЗ этого заказчика.

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

Возвращаясь к теме тикета, мне кажется более чем странным, если внешний вид оформления окна (к которому относится его заголовок) определяет человек, который никогда на это оформление и смотреть-то не будет (который всего лишь продает/покупает нашу систему диспетчерской связи), а не тот пользователь, который реально с ней работает.

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

Заметь, если мы оставим оформлением окна window manager'у, у "заказчика" вместе с пользователем будет возможность сделать заголовки такими, какие они хотят - с черным цветом на белом фоне. Мы ему этого не запрещаем. Убирая же "штатный" заголовок и пририсовывая свой, мы тем самым отбираем у них возможность изменить то, что "забито гвоздями" в коде программы. Мне кажется, это плохо...

Last edited 8 years ago by alx (previous) (diff)

comment:7 by san, 8 years ago

Немного уточню: заказчик для нас, в данном случае - МВтел(в лице Вани, менеджера, ответственного за этот продукт). Клиенты - пока несколько УМГ Белтрансгаза. И Мвтел, как раз, намного ближе к клиентам чем мы.

По теме тикета: заказчик настаивает на "забитом гвоздями" хеадере и стиле всех окон.
Алексей, я понимаю что этим мы даём пользователю некоторые ограничения свободы, но вынужден принять пожелания заказчика.

Алексей, кроме того, что хеадер забит гвоздями и не подчиняется настройкам системы, какие ещё ты видишь недостатки у нового хеадера?

  1. Не позволяет перемещать окно мышью за заголовок.
  2. ...

in reply to:  7 comment:8 by alx, 8 years ago

Resolution: invalid
Status: newclosed

Replying to san:

Немного уточню: заказчик для нас, в данном случае - МВтел(в лице Вани, менеджера, ответственного за этот продукт).

OK. Видимо, я просто был не в курсе. Раз есть заказчик, значит есть договори ТЗ, и мы должны выполнять, что там записано.

Алексей, кроме того, что хеадер забит гвоздями и не подчиняется настройкам системы, какие ещё ты видишь недостатки у нового хеадера?

  1. Не позволяет перемещать окно мышью за заголовок.
  2. ...

Это невозможно сформулировать по пунктам. Недостаток, фактически, один - заголовок не управляется Window-Manager'ом. Все остальное (как, например, невозможность перемещения окна за заголовок) - лишь следствия этого прискорбного факта. И пытаться предугадать их все бессмысленно, ибо все эти функции настраиваются пользователем. Добавит завтра пользователь на заголовок, например, кнопочку "скриншот окна", и появится у нас очередной, 123-й недостаток: у всех окон есть кнопочка "скриншот", а у нашего - нет...

Впрочем, могу сформулировать еще один "глобальный" недостаток: наш "самопальный" заголовок - некрасивый, и его стиль не соответствует стилю всех прочих окон. Впрочем, это уже вкусовщина...

comment:9 by san, 8 years ago

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

Но я, честно говоря, до этого диалога, над недостатками такого решения не сильно задумывался. Почитав аргументы Алексея, считаю что он прав и делать свой хеадер не стоит.
Еще раз обсудил эту проблему с Ваней.

Итог: Возвращаем нормальные хеадеры для окон.

comment:10 by san, 8 years ago

Resolution: invalid
Status: closedreopened

comment:11 by dimag, 8 years ago

Resolution: fixed
Status: reopenedclosed

comment:13 by alx, 8 years ago

Resolution: fixed
Status: closedreopened

В окне информации о пользователе хереда по-прежнему нет.

comment:14 by dimag, 8 years ago

Что ты думаешь Александр, надо ли для заголовки окон получения информации о пользователи, окна редактирования пользователя сделать такими же как заголовок окна входа пользователя в систему?

comment:15 by san, 8 years ago

Там пару коментов выше ведь мой ответ есть, жирным шрифтом выделен

Итог: Возвращаем нормальные хеадеры для окон.

comment:16 by dimag, 8 years ago

Resolution: fixed
Status: reopenedclosed

comment:17 by san, 6 years ago

Milestone: Текущее2 очередь

Milestone renamed

Note: See TracTickets for help on using tickets.