-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make lilka::Menu component avaliable for reuse #51
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Супер, є лише декілька дрібних зауважень
sdk/lib/lilka/src/lilka/ui.h
Outdated
@@ -81,8 +81,19 @@ class Menu { | |||
/// | |||
/// Якщо пункт обрано (користувач натиснув кнопку "A"), повертається ``true``, інакше ``false``. Після виклику цієї функції пункт перестає бути обраним. | |||
bool isFinished(); | |||
/// Дозволяє змінити пункту меню |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Опечатка, + краще в наказовому стилі: "Змінити пункт меню"
І ще, можна додати вже й getItem
, раз вже є всі інші методи :)
sdk/lib/lilka/src/lilka/ui.h
Outdated
/// @param color Колір пункту. За замовчуванням ``0`` (стандартний колір). | ||
/// @param postfix Текст, який додається після заголовка пункту і вирівнюється до правого краю меню. | ||
/// | ||
/// Повертає значення true якщо пункт було змінено |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Бракує коми перед "якщо"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Супер, є тільки одне зауваження (див. комент)
menu->clearItems and menu->setItem implementation