Applications
/
ubl-settings-video
10
0
Fork 2
Code Issues 5 Pull Requests Packages Projects Releases Wiki Activity
Labels Milestones

master #6

Merged
asmeron merged 19 commits from YanTheKaller/ubl-settings-video:master into develop 3 years ago
Conversation 3 Commits 19 Files Changed 22
YanTheKaller commented 3 years ago
Collaborator
There is no content yet.
YanTheKaller added 7 commits 3 years ago
YanTheKaller commented 3 years ago
Poster
Collaborator

Пока не принимайте, пока делал скриншоты обнаружил несколько кнопок без локализации

Пока не принимайте, пока делал скриншоты обнаружил несколько кнопок без локализации
image.png
101 KiB
image.png
20 KiB
image.png
14 KiB
asmeron reviewed 3 years ago
source/ubl-settings-video.c Outdated
g_signal_connect(G_OBJECT(widgets->mainHeaderSettingsAboutMenuItem), "activate", G_CALLBACK(on_about), NULL);
gtk_combo_box_text_append_text(GTK_COMBO_BOX_TEXT(widgets->mainDriverProprietaryDriverCombo),_("Don't use"));
asmeron commented 3 years ago
Owner

Уточнить у Игоря, почему он перешёл с функции локализации _
на прямой вызов gettext

Уточнить у Игоря, почему он перешёл с функции локализации _ на прямой вызов gettext
YanTheKaller commented 3 years ago
Poster
Collaborator

Он не знал как сделать так чтобы команда генерации pot файла искала _() вместе с gettext(). Я показал

Он не знал как сделать так чтобы команда генерации pot файла искала _() вместе с gettext(). Я показал
asmeron requested changes 3 years ago
source/ubl-settings-video.h Outdated
#define load_drivers_global_command "/usr/bin/ubconfig --default --source global get video FAILSAFENVIDIA FAILSAFEATI OPTIRUN PRIMUSRUN "
#define save_drivers_global_command "/usr/bin/ubconfig --default --source global set video FAILSAFENVIDIA=%s FAILSAFEATI=%s OPTIRUN=%s PRIMUSRUN=%s "
asmeron commented 3 years ago
Owner

у ubconfig при методе set не использовать --default и заменить --source на --target

А если это указано в ТЗ, то отписаться Максиму в группе о некорректности указанных команд в ТЗ

у ubconfig при методе set не использовать --default и заменить --source на --target А если это указано в ТЗ, то отписаться Максиму в группе о некорректности указанных команд в ТЗ
YanTheKaller commented 3 years ago
Poster
Collaborator

В ТЗ команд не указано, я их всё ещё не доделал, они в коде пока что не используются

В ТЗ команд не указано, я их всё ещё не доделал, они в коде пока что не используются
asmeron commented 3 years ago
Owner

Тогда как заметку используй

Тогда как заметку используй
source/ubl-settings-video.h Outdated
#define load_drivers_local_command "/usr/bin/ubconfig --default --source system get video FAILSAFENVIDIA FAILSAFEATI OPTIRUN PRIMUSRUN "
#define save_drivers_local_command "/usr/bin/ubconfig --default --source system set video FAILSAFENVIDIA=%s FAILSAFEATI=%s OPTIRUN=%s PRIMUSRUN=%s "
asmeron commented 3 years ago
Owner

у ubconfig при методе set не использовать --default и заменить --source на --target

А если это указано в ТЗ, то отписаться Максиму в группе о некорректности указанных команд в ТЗ

Научить Игоря, по использованию макроса описывающего в директиве внешнюю команду, т.к. Игорь использует внутри кода, и часто делает ошибки !

у ubconfig при методе set не использовать --default и заменить --source на --target А если это указано в ТЗ, то отписаться Максиму в группе о некорректности указанных команд в ТЗ Научить Игоря, по использованию макроса описывающего в директиве внешнюю команду, т.к. Игорь использует внутри кода, и часто делает ошибки !
YanTheKaller commented 3 years ago
Poster
Collaborator

#6 Ответил выше

https://gitea.ublinux.ru/Applications/ubl-settings-video/pulls/6#issuecomment-2607 Ответил выше
ubl-settings-video.desktop Outdated
[Desktop Entry]
Encoding=UTF-8
Name=ubl-settings-video
Name[ru]=Настройки Видеовывода
asmeron commented 3 years ago
Owner

Заменить на:
Настройки видеоадаптера и дисплея

Заменить на: Настройки видеоадаптера и дисплея
YanTheKaller commented 3 years ago
Poster
Collaborator

f4a60098e0
Исправил здесь

https://gitea.ublinux.ru/YanTheKaller/ubl-settings-video/commit/f4a60098e04f07676a588919ec74258e0eaccf8a Исправил здесь
ubl-settings-video.desktop Outdated
Name=ubl-settings-video
Name[ru]=Настройки Видеовывода
GenericName=ubl-settings-video
GenericName[ru]=Настройки Видеовывода
asmeron commented 3 years ago
Owner

Заменить на:
Настройки видеоадаптера и дисплея

Заменить на: Настройки видеоадаптера и дисплея
YanTheKaller commented 3 years ago
Poster
Collaborator

f4a60098e0
Исправил здесь

https://gitea.ublinux.ru/YanTheKaller/ubl-settings-video/commit/f4a60098e04f07676a588919ec74258e0eaccf8a Исправил здесь
asmeron commented 3 years ago
Owner

Исправил не корректно, в предложении должно быть только первое слово с большой буквы !

Исправил не корректно, в предложении должно быть только первое слово с большой буквы !
YanTheKaller commented 3 years ago
Poster
Collaborator

8382fb0914

https://gitea.ublinux.ru/YanTheKaller/ubl-settings-video/commit/8382fb0914e0db11047b7278020df23403e6025e
ubl-settings-video.desktop Outdated
Name[ru]=Настройки Видеовывода
GenericName=ubl-settings-video
GenericName[ru]=Настройки Видеовывода
Comment=ubl-settings-video
asmeron commented 3 years ago
Owner

Заменить на:
Application for managing video adapter and display settings

Заменить на: Application for managing video adapter and display settings
YanTheKaller commented 3 years ago
Poster
Collaborator

f4a60098e0
Исправил здесь

https://gitea.ublinux.ru/YanTheKaller/ubl-settings-video/commit/f4a60098e04f07676a588919ec74258e0eaccf8a Исправил здесь
asmeron commented 3 years ago
Owner

Исправил не корректно, в предложении должно быть только первое слово с большой буквы !

Исправил не корректно, в предложении должно быть только первое слово с большой буквы !
YanTheKaller commented 3 years ago
Poster
Collaborator

8382fb0914

https://gitea.ublinux.ru/YanTheKaller/ubl-settings-video/commit/8382fb0914e0db11047b7278020df23403e6025e
ubl-settings-video.desktop Outdated
GenericName=ubl-settings-video
GenericName[ru]=Настройки Видеовывода
Comment=ubl-settings-video
Comment[ru]=Приложение для управления настройками видеовывода для UBLinux
asmeron commented 3 years ago
Owner

Заменить на:
Приложение для управления настройками видеоадаптера и дисплея

Заменить на: Приложение для управления настройками видеоадаптера и дисплея
YanTheKaller commented 3 years ago
Poster
Collaborator

f4a60098e0
Исправил здесь

https://gitea.ublinux.ru/YanTheKaller/ubl-settings-video/commit/f4a60098e04f07676a588919ec74258e0eaccf8a Исправил здесь
YanTheKaller commented 3 years ago
Poster
Collaborator

Пока не принимайте, пока делал скриншоты обнаружил несколько кнопок без локализации

Исправил в 58f8eb3cde

> Пока не принимайте, пока делал скриншоты обнаружил несколько кнопок без локализации Исправил в https://gitea.ublinux.ru/YanTheKaller/ubl-settings-video/commit/58f8eb3cde47f7cdb474b258f5270d4aa1c835e3
asmeron requested changes 3 years ago
asmeron left a comment
Owner

Аналогичные ошибки исправить во всех своих проектах !

Аналогичные ошибки исправить во всех своих проектах !
README.md Outdated
# ubl-settings-video
# Видеокарта/Экран
asmeron commented 3 years ago
Owner

Исправить название на корректное

Исправить название на корректное
YanTheKaller commented 3 years ago
Poster
Collaborator

8382fb0914

https://gitea.ublinux.ru/YanTheKaller/ubl-settings-video/commit/8382fb0914e0db11047b7278020df23403e6025e
source/ubl-settings-video.h Outdated
#define icon_name "com.ublinux.ubl-settings-video"
#define banner_path "/usr/share/ubl-settings-video/ui/ubl-settings-video-banner.png"
asmeron commented 3 years ago
Owner

Все изображения должны быть в каталоге /usr/share/ubl-settings-video/images/
Исправь во всех своих проектах !

Все изображения должны быть в каталоге /usr/share/ubl-settings-video/images/ Исправь во всех своих проектах !
YanTheKaller commented 3 years ago
Poster
Collaborator

d37a44bd2e
Изменил пути иконки
0d78edd63a
Исправил пути баннера
30cc4f3232
Исправил Makefile

https://gitea.ublinux.ru/YanTheKaller/ubl-settings-video/commit/d37a44bd2ef8d0b9060b9aad62d95d0f2d9a854d Изменил пути иконки https://gitea.ublinux.ru/YanTheKaller/ubl-settings-video/commit/0d78edd63a31a4ab4c2cebc6f841f5843de6db6c Исправил пути баннера https://gitea.ublinux.ru/YanTheKaller/ubl-settings-video/commit/30cc4f3232df461192eafb2ffac084cf5d45d0d4 Исправил Makefile
ubl-settings-video.pot Outdated
# SOME DESCRIPTIVE TITLE.
asmeron commented 3 years ago
Owner

Заголовок проекта, так-же должен быть заполнен реквизитами соответствующими !

Заголовок проекта, так-же должен быть заполнен реквизитами соответствующими !
YanTheKaller commented 3 years ago
Poster
Collaborator

b547aeb869

https://gitea.ublinux.ru/YanTheKaller/ubl-settings-video/commit/b547aeb869c8649ada43f65aae9d2cf4670e3774
user-166 commented 3 years ago
Collaborator

Проверил.

Проверил.
asmeron merged commit dee07c5541 into develop 3 years ago

Reviewers

asmeron requested changes 3 years ago
The pull request has been merged as dee07c5541.
Sign in to join this conversation.
Reviewers
Request review
No reviewers
asmeron
Labels
Apply labels
Clear labels
вопрос
Требуется дополнительная информация дизайн
Проблема с дизайном дубликат
Эта проблема или запрос на включение уже существует неисправимо
Это не будет исправлено неисправно
Что-то неисправно ошибка
Что-то не работает помощь
Нужна помощь улучшение
Новая возможность
No Label вопрос дизайн дубликат неисправимо неисправно ошибка помощь улучшение
Milestone
Set milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Applications/ubl-settings-video#6
Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch 'YanTheKaller/ubl-settings-video:master'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes