master #76

Merged
asmeron merged 28 commits from :master into develop 3 years ago
Igor1 commented 3 years ago

Добавлено в окно сообщений об ошибках в каком поле ошибка.

Добавлено в окно сообщений об ошибках в каком поле ошибка.
Igor1 added 8 commits 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 2 commits 3 years ago
asmeron requested changes 3 years ago
asmeron left a comment
Owner

Мелкие ошибки исправить, а по иконкам

  • checked.svg
  • warning.svg
    решить, что делать с ними глобально !
Мелкие ошибки исправить, а по иконкам - checked.svg - warning.svg решить, что делать с ними глобально !
Makefile Outdated
@install -Dm644 -t "${pkgdir}/usr/share/${pkgname}/images/" bg_top.png
@install -Dm644 -t "${pkgdir}/usr/share/${pkgname}/images/" checked.svg
Owner

Тут они точно нужны ??

Почему вообще системные, зависящие от темя не использовать ?

Тут они точно нужны ?? Почему вообще системные, зависящие от темя не использовать ?
Collaborator

Потому что эти нарисовали сами и они хорошо вписаны в интерфейс. Про баннер - без комментариев.

Потому что эти нарисовали сами и они хорошо вписаны в интерфейс. Про баннер - без комментариев.
Owner

Хорошо, если заменять не будем на системные, тогда доступ правильный сделать !

Хорошо, если заменять не будем на системные, тогда доступ правильный сделать !
Igor1 commented 3 years ago
Poster

Удалил, теперь иконки грузятся только в /usr/share/icons/hicolor/scalable/status/.
С такими именами {ubl-datetime-checked.svg,ubl-datetime-warning.svg}

Удалил, теперь иконки грузятся только в /usr/share/icons/hicolor/scalable/status/. С такими именами {ubl-datetime-checked.svg,ubl-datetime-warning.svg}
Makefile Outdated
@install -Dm644 -t "${pkgdir}/usr/share/${pkgname}/images/" bg_top.png
@install -Dm644 -t "${pkgdir}/usr/share/${pkgname}/images/" checked.svg
@install -Dm644 -t "${pkgdir}/usr/share/${pkgname}/images/" warning.svg
Owner

Тут они точно нужны ??

Почему вообще системные, зависящие от темя не использовать ?

Тут они точно нужны ?? Почему вообще системные, зависящие от темя не использовать ?
Collaborator

Потому что эти нарисовали сами и они хорошо вписаны в интерфейс. Про баннер - без комментариев.

Потому что эти нарисовали сами и они хорошо вписаны в интерфейс. Про баннер - без комментариев.
Owner

Зачем их хранить в двух местах ?

Зачем их хранить в двух местах ?
Igor1 commented 3 years ago
Poster

Удалил, теперь иконки грузятся только в /usr/share/icons/hicolor/scalable/status/

Удалил, теперь иконки грузятся только в /usr/share/icons/hicolor/scalable/status/
Makefile Outdated
@install -Dm644 -t "${pkgdir}/usr/share/${pkgname}/images/" bg_top.png
@install -Dm644 -t "${pkgdir}/usr/share/${pkgname}/images/" checked.svg
@install -Dm644 -t "${pkgdir}/usr/share/${pkgname}/images/" warning.svg
@install -Dm644 -t "${pkgdir}/usr/share/icons/hicolor/scalable/status/" "checked.svg"
Owner

Для чего хранить в двух местах эти иконки ?:

  • /usr/share/${pkgname}/images/
  • /usr/share/icons/hicolor/scalable/status/

Имена иконок дать правильные, т.к. эти имена совпадают с существующими !
Для примера заменить на: ru.ublinux.ubl-settings-datetime.checked.svg

Для чего хранить в двух местах эти иконки ?: - /usr/share/${pkgname}/images/ - /usr/share/icons/hicolor/scalable/status/ Имена иконок дать правильные, т.к. эти имена совпадают с существующими ! Для примера заменить на: ru.ublinux.ubl-settings-datetime.checked.svg
Igor1 commented 3 years ago
Poster

Удалил, теперь иконки грузятся только в /usr/share/icons/hicolor/scalable/status/

Удалил, теперь иконки грузятся только в /usr/share/icons/hicolor/scalable/status/
Makefile Outdated
@install -Dm644 -t "${pkgdir}/usr/share/${pkgname}/images/" checked.svg
@install -Dm644 -t "${pkgdir}/usr/share/${pkgname}/images/" warning.svg
@install -Dm644 -t "${pkgdir}/usr/share/icons/hicolor/scalable/status/" "checked.svg"
@install -Dm644 -t "${pkgdir}/usr/share/icons/hicolor/scalable/status/" "warning.svg"
Owner

Для чего хранить в двух местах эти иконки ?:

  • /usr/share/${pkgname}/images/
  • /usr/share/icons/hicolor/scalable/status/

Имена иконок дать правильные, т.к. эти имена совпадают с существующими !
Для примера заменить на: ru.ublinux.ubl-settings-datetime.warning.svg

Для чего хранить в двух местах эти иконки ?: - /usr/share/${pkgname}/images/ - /usr/share/icons/hicolor/scalable/status/ Имена иконок дать правильные, т.к. эти имена совпадают с существующими ! Для примера заменить на: ru.ublinux.ubl-settings-datetime.warning.svg
Igor1 commented 3 years ago
Poster

Исправил.

5e022ea002

Исправил. https://gitea.ublinux.ru/Igor1/ubl-settings-datetime/commit/5e022ea002d06b07551b090055e820ccb14dd33d
Owner

Внимательнее перечитывай сообщение, перед публикацией ответа !
Имя должно быть формата: ru.ublinux.ubl-settings-datetime.warning.svg

Внимательнее перечитывай сообщение, перед публикацией ответа ! Имя должно быть формата: ru.ublinux.ubl-settings-datetime.warning.svg
Igor1 commented 3 years ago
Poster

Исправил
cabfa962a6

Исправил https://gitea.ublinux.ru/Igor1/ubl-settings-datetime/commit/cabfa962a62682c29db3f01ea31e582ffa944ac1
Igor1 commented 3 years ago
Poster

Исправил как решили, по новому.
f0b9702e41
b725966525

Исправил как решили, по новому. https://gitea.ublinux.ru/Igor1/ubl-settings-datetime/commit/f0b9702e41a469a2c7d52ccd5902908a1ab4108a https://gitea.ublinux.ru/Igor1/ubl-settings-datetime/commit/b7259665254364a3294ebe724936c79b04233a44
btnBoxAboutDialog->set_visible(false);
this->event();
imgInfo->set("/usr/share/ubl-settings-datetime/images/checked.svg");
imgInfo->set("/usr/share/icons/hicolor/scalable/status/warning.svg");
Owner

Я уже говорил, для иконок не использовать прямой путь, сколько повторять одно и то-же !

Я уже говорил, для иконок не использовать прямой путь, сколько повторять одно и то-же !
Igor1 commented 3 years ago
Poster

Исправил.
d67109c053

Исправил. https://gitea.ublinux.ru/Igor1/ubl-settings-datetime/commit/d67109c0539025c1a2498b914c062a304c137152
imgInfo->set("/usr/share/ubl-settings-datetime/images/checked.svg");
imgInfo->set("/usr/share/icons/hicolor/scalable/status/warning.svg");
info_status_app("boxInfoMessError");
lblWarning->set_text(_("The program must be run with superuser privileges!"));
Owner

superuser ??
заменить на:
The program must be run as root

superuser ?? заменить на: The program must be run as root
Igor1 commented 3 years ago
Poster

Перевод исправил.
5e022ea002

Перевод исправил. https://gitea.ublinux.ru/Igor1/ubl-settings-datetime/commit/5e022ea002d06b07551b090055e820ccb14dd33d
builder->get_widget("imgDonwload", imgDonwload);
Owner

Правильно пишется: Download

Исправить везде !

Правильно пишется: Download Исправить везде !
Igor1 commented 3 years ago
Poster

Исправил
20ec428af7

Исправил https://gitea.ublinux.ru/Igor1/ubl-settings-datetime/commit/20ec428af7639eb527464aa05e2f3d0d204cc38d
this->fill_in_reg_zone(cmd_zone);
string hw = "/usr/bin/ubconfig --default --source global get clock HWCLOCK_SYNC";
string hw = "/usr/bin/ubconfig --source global get clock HWCLOCK_SYNC";
string hw_default = "/usr/bin/ubconfig --default --source global get clock HWCLOCK_SYNC";
Owner

Для чего переменная hw и string hw_default одновременно существуют ??

Если нужно реально дефолтное значение только, то команда другая /usr/bin/ubconfig --source default get clock HWCLOCK_SYNC

И почему не использовать опцию --raw ??
/usr/bin/ubconfig --raw --source default get clock HWCLOCK_SYNC

Для чего переменная hw и string hw_default одновременно существуют ?? Если нужно реально дефолтное значение только, то команда другая /usr/bin/ubconfig --source default get clock HWCLOCK_SYNC И почему не использовать опцию --raw ?? /usr/bin/ubconfig --raw --source default get clock HWCLOCK_SYNC
Igor1 commented 3 years ago
Poster

И почему не использовать опцию --raw ??
/usr/bin/ubconfig --raw --source default get clock HWCLOCK_SYNC

Потому-что надо будет переделывать, работу оболочки в двух утилитах. И как проверять на ошибки, если знака "=" нет ?. Лучше уже не использовать raw.

И почему не использовать опцию --raw ?? /usr/bin/ubconfig --raw --source default get clock HWCLOCK_SYNC Потому-что надо будет переделывать, работу оболочки в двух утилитах. И как проверять на ошибки, если знака "=" нет ?. Лучше уже не использовать raw.
Igor1 commented 3 years ago
Poster

Исправил команду
3468ec22d0

Исправил команду https://gitea.ublinux.ru/Igor1/ubl-settings-datetime/commit/3468ec22d01467087a1fb69dbb633cc29e69edcb
Owner

Для чего переменная hw и string hw_default одновременно существуют ??

Ответ не дан, исправления не вижу !

> Для чего переменная hw и string hw_default одновременно существуют ?? Ответ не дан, исправления не вижу !
Igor1 commented 3 years ago
Poster

Исправил.
fd671140bb
5dc590b4b3

Исправил. https://gitea.ublinux.ru/Igor1/ubl-settings-datetime/commit/fd671140bb8b81f3f6af9c14d3ea49f79dddd006 https://gitea.ublinux.ru/Igor1/ubl-settings-datetime/commit/5dc590b4b347e3384e46814859e29904ef714b9e
if (error_info > 0 || warning_info > 0) {
info_status_app("boxInfoMessError");
imgInfo->set("/usr/share/ubl-settings-datetime/images/warning.svg");
imgInfo->set("/usr/share/icons/hicolor/scalable/status/warning.svg");
Owner

Я уже говорил, для иконок не использовать прямой путь, сколько повторять одно и то-же !

Я уже говорил, для иконок не использовать прямой путь, сколько повторять одно и то-же !
Igor1 commented 3 years ago
Poster

Исправил.
d67109c053

Исправил. https://gitea.ublinux.ru/Igor1/ubl-settings-datetime/commit/d67109c0539025c1a2498b914c062a304c137152
else {
info_status_app("boxInfoMessOK");
imgInfo->set("/usr/share/ubl-settings-datetime/images/checked.svg");
imgInfo->set("/usr/share/icons/hicolor/scalable/status/checked.svg");
Owner

Я уже говорил, для иконок не использовать прямой путь, сколько повторять одно и то-же !

Я уже говорил, для иконок не использовать прямой путь, сколько повторять одно и то-же !
Igor1 commented 3 years ago
Poster
https://gitea.ublinux.ru/Igor1/ubl-settings-datetime/commit/d67109c0539025c1a2498b914c062a304c137152
msgid "The program must be run with \nsuperuser privileges!"
msgstr "Программа должна быть запущена с правами \nсуперпользователя!"
msgid "The program must be run with superuser privileges!"
Owner

Исправить как выше писал, и перевод

Исправить как выше писал, и перевод
Igor1 commented 3 years ago
Poster

Перевод исправил.
5e022ea002

Перевод исправил. https://gitea.ublinux.ru/Igor1/ubl-settings-datetime/commit/5e022ea002d06b07551b090055e820ccb14dd33d
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 2 commits 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
asmeron reviewed 3 years ago
}
void MainWindow::settings() {
map_error["xdg-open https://wiki.ublinux.ru/ru/home"] = string(_(": \"xdg-open\""));
Owner

Почему https://wiki.ublinux.ru/ru/home не через локализацию ??

Почему `https://wiki.ublinux.ru/ru/home` не через локализацию ??
asmeron merged commit e7e27ec2ae into develop 3 years ago

Reviewers

asmeron requested changes 3 years ago
The pull request has been merged as e7e27ec2ae.
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Applications/ubl-settings-datetime#76
Loading…
There is no content yet.