sadr0b0t / vkurse

Automatically exported from code.google.com/p/vkurse
0 stars 0 forks source link

Problems with user web-interface #11

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
1. На странице http://vkurse.innolab.net.ru:8180/vkurse/ в 
верхнем правом углу у меня отображаются 
непонятные символы "п»ї"

2. На странице http://vkurse.innolab.net.ru:8180/vkurse/ на мой 
взгляд не хватает выбора курса, как это 
сделано на http://vkurse.innolab.net.ru:8180/vkurse/welcome.jsp

Поскольку сейчас разрабатывается новый 
дизайн интерфейса пользователя замечание 2 
незначительно.

Original issue reported on code.google.com by andrej.k...@gmail.com on 4 Nov 2010 at 7:46

GoogleCodeExporter commented 9 years ago
1. Пофикшено.

2. На самом деле, курса и не должно быть. У 
нас в базе - только список групп. А курс - 
параметр. Фильтровать группы по курсу без 
javascript невозможно.

Original comment by Lockyw...@gmail.com on 4 Nov 2010 at 10:41

GoogleCodeExporter commented 9 years ago
Сделал для бага реопен по второму пункту:

>2. На самом деле, курса и не должно быть. У 
нас в базе - только список групп. А курс - 
параметр. Фильтровать группы по курсу без 
javascript невозможно.

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

791 (курс 4)
792 (курс 4)
798 (курс 3)
...

или

курс4: 791
курс4: 792
курс3: 798
...

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

Original comment by bender...@gmail.com on 4 Nov 2010 at 11:47

GoogleCodeExporter commented 9 years ago
>>Сделал для бага реопен по второму пункту:

Зачем? Везде, где можно - курс и так выкинут.
Писать его рядом с группой - 
нецелесообразно. Каждый сам помнит, какой 
курс он учится. Полезной информации не 
несет, а работы добавляет - каждый год 
менять в базе курс на следующий. Это не 
нужно.

Закрывайте баг.

Original comment by Lockyw...@gmail.com on 4 Nov 2010 at 5:03

GoogleCodeExporter commented 9 years ago
>Зачем? Везде, где можно - курс и так выкинут.
>Писать его рядом с группой - 
нецелесообразно. Каждый сам помнит, какой 
курс он 
>учится. >Полезной информации не несет, а 
работы добавляет - каждый год менять в базе
>курс на >следующий. Это не нужно.

Речь об админке:
http://vkurse.innolab.net.ru:8180/vkurse/admin/admin_entrance.jsp

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

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

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

>Закрывайте баг.

Убери комбобокс с курсом - можно будет 
закрывать.

P.S. кстати, кстати - а может быть мы не будем 
скрывать поле "курс", а просто переименуем 
его в интерфейсе (даже API/базу данных 
трогать не будем) в "год создания" - типа 
группа 792 (2007), 798 (2008) и т.п. - в этом случае оно 
действительно могло бы быть полезным и 
каждый год ничего переименовывать не 
нужно. Или у вас в номере группы уже 
зашифрован этот год? Если да, то поле "курс" 
действительно лишнее в любом виде и нужно 
его скрывать отовсюду. Какие будут 
комментарии?

Original comment by bender...@gmail.com on 5 Nov 2010 at 12:13

GoogleCodeExporter commented 9 years ago
Убрал откуда увидел.

Original comment by Lockyw...@gmail.com on 7 Nov 2010 at 3:33