MPC-BE forum

MPC-BE => Баг Репорт / The bug report => Тема начата: Evgeniy1990 от 01 марта 2016, 17:42:02

Название: Регулятор громкости - белая полоска (баг-поломка после r1069)
Отправлено: Evgeniy1990 от 01 марта 2016, 17:42:02
Bug after r1069

Раньше у нас был такой баг, когда при переключении со светлой темы на темную, появлялась белая полоска снизу регулятора громкости. Он был исправлен в свое время.

Обратите пожалуйста внимание на скриншот: https://yadi.sk/i/jA5e-CYGpoRZN

Но теперь этот баг снова вернулся, т.е. произошла поломка уже исправленного бага.
Другими словами, поломали то, что уже было исправлено.

Произошла она именно после внедрения изменений в r1069.

Диапазон поиска: [1067 -> 1072].

Исходя из списка изменений, входящих в данный диапазон, единственное изменение, которое вызвало поломку, - это изменение под номером r1069.
Название: Регулятор громкости - белая полоска (баг-поломка после r1069)
Отправлено: Aleksoid1978 от 02 марта 2016, 01:55:07
Видимо такое происходит только на WinXP, т.к. у себя воспроизвести не могу. Или может есть какой-то хитрый алгоритм ??
Название: Регулятор громкости - белая полоска (баг-поломка после r1069)
Отправлено: Evgeniy1990 от 02 марта 2016, 05:36:46
Цитата: Aleksoid1978Видимо такое происходит только на WinXP, т.к. у себя воспроизвести не могу. Или может есть какой-то хитрый алгоритм ??
Да, совершенно верно, такой баг проявляется только на Windows XP.

Алгоритм воспроизведения простой:

1. Открыть "Настройки" -> "Интерфейс" -> Отключить опцию "Использовать "темную" тему оформления (по умолчанию)", применить изменения.
2. Повторно открыть: "Настройки" -> "Интерфейс" -> Включить опцию "Использовать "темную" тему оформления (по умолчанию)", применить изменения.

Фактический результат: после переключения с классической темы обратно на темную тему, появляется эта белая полоска под регулятором громкости.
Ожидаемый результат: после переключения с классической темы обратно на темную тему, не должно быть этой белой полоски под регулятором громкости.
Название: Регулятор громкости - белая полоска (баг-поломка после r1069)
Отправлено: Aleksoid1978 от 02 марта 2016, 06:03:32
Понятно. Ну да ладно - в этом ничего криминального нету :)
Название: Регулятор громкости - белая полоска (баг-поломка после r1069)
Отправлено: Evgeniy1990 от 02 марта 2016, 06:16:34
Цитата: Aleksoid1978Понятно. Ну да ладно - в этом ничего криминального нету :)
Факт в том, что это поломка. Причем раньше вы этот баг уже правили.
Она возникла после того, как вы добавили изменения в r1069.
Посмотрите и сравните пожалуйста код, как было раньше и что сейчас, может быть сможете это поправить?
Название: Регулятор громкости - белая полоска (баг-поломка после r1069)
Отправлено: Aleksoid1978 от 02 марта 2016, 06:26:34
Ну и что - ну поломка. Но нифига не критичная. И уже тем более обычный пользователь не будет переключать темную/не темную тему. По коду нельзя понять что происходит.
Название: Регулятор громкости - белая полоска (баг-поломка после r1069)
Отправлено: Evgeniy1990 от 02 марта 2016, 06:47:17
Цитата: Aleksoid1978Ну и что - ну поломка. Но нифига не критичная. И уже тем более обычный пользователь не будет переключать темную/не темную тему. По коду нельзя понять что происходит.
А если попробовать через отладочный билд с лог-файлом?
Название: Регулятор громкости - белая полоска (баг-поломка после r1069)
Отправлено: Aleksoid1978 от 02 марта 2016, 12:06:04
Нет, такое логами хрен отловить.
Название: Регулятор громкости - белая полоска (баг-поломка после r1069)
Отправлено: Evgeniy1990 от 22 апреля 2016, 09:01:43
Цитата: Aleksoid1978Нет, такое логами хрен отловить.
Просто интересно, а как вы раньше отлавливали этот баг и правили его?
Может быть вы сможете вспомнить и по аналогии сделать тоже самое сейчас, что и раньше?

Насколько я понял, дело в обновлении неких переменных "на лету".
Они просто почему-то не обновляются, или обновляются, но некорректно.

Ведь, если закрыть плеер и снова его запустить, то этого бага не будет.
Получается что обновление неких переменных происходит корректно только после перезапуска самой программы, хотя по логике должно корректно выполняться на лету.
[merge_posts_bbcode]Добавлено: 2016-04-22 09:01:43[/merge_posts_bbcode]

Тема: пропадание измененных перерисованных иконок.

И снова эти проклятые изменения в SVN r1069.

Обнаружил на чистой XP-системе, что при определенных действиях, могут пропадать иконки, особенно значок "GPU".
Баг-поломка проявляется в случае действия - "наложение", например таким образом:

1. Открыть видео файл
2. Открыть его свойства

Фактический результат: видна только правая часть иконки "GPU"

3. Развернуть окно на весь экран
4. Обратно свернуть в оконный режим

Фактический результат: уже полностью исчез весь значок "GPU"

В текущей сессии его уже никак не вернуть.

Мда, полнейшее безобразие.  У меня такое чувство, что в этих изменениях вообще не были учтены определенные спецификации (критерии) для Windows XP.

Если подобрать "особые видео файлы", то можно вообще очень часто наблюдать этот изврат со значками - иконками и GUI в целом.

Лично я не одобряю эти изменения в SVN r1069, ибо они принесли больше вреда, чем пользы.
Поэтому прошу вас, пожалуйста внесите корректировки в данные изменения, или хоть как-то их переделайте.

Это не дело - так ломать GUI.
Название: Регулятор громкости - белая полоска (баг-поломка после r1069)
Отправлено: Aleksoid1978 от 22 апреля 2016, 09:33:52
Да не надо "на лету" менять темное оформление на стандартное, вот и все. Понятно для тестов, для нахождения багов - это одно. Рядовой пользователь такое уж точно делать не будет, в этом я уверен на 99.9% :)

Второй момент - WinXP. Закрываем тему ... сразу же.
[merge_posts_bbcode]Добавлено: 2016-04-22 16:33:52[/merge_posts_bbcode]

Добавлю - вот реально мне(да и v0lt) будет намного проще полностью отказаться от поддержки WinXP, чем из-за раза в раз слушать про это.
Название: Регулятор громкости - белая полоска (баг-поломка после r1069)
Отправлено: Evgeniy1990 от 22 апреля 2016, 09:35:28
Цитата: Aleksoid1978Да не надо "на лету" менять темное оформление на стандартное, вот и все. Понятно для тестов, для нахождения багов - это одно. Рядовой пользователь такое уж точно делать не будет, в этом я уверен на 99.9% :)

Тем более - WinXP. Закрываем тему.
[merge_posts_bbcode]Добавлено: 2016-04-22 16:33:52[/merge_posts_bbcode]

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

Вы этими изменениями поломали работу иконок на темной теме.
Название: Регулятор громкости - белая полоска (баг-поломка после r1069)
Отправлено: Aleksoid1978 от 22 апреля 2016, 09:47:11
Ключевое слово - Windows XP, так что тему закрыли.