UI-DEN-3253_UI Alarm Design

Activity

UI-DEN-3253-1 244

Keyboard shortcuts  
  • Summarize the review outcomes (optional)
     
    #permalink

    Details

    Warning: no files are visible, they have all been filtered.
    Participant Role Time Spent Comments Latest Comment
    plucia  (deleted user)
    Author & Moderator 7h 20m 90 Done
    Reviewer - Complete 8h 57m 107 RESOLVED
    pmontazemi  (deleted user)
    Reviewer completed
    Reviewer - Complete 1h 18m 46 RESOLVED.
    Reviewer - 0% reviewed      
    Reviewer - 0% reviewed      
    Reviewer - 0% reviewed   1 RESOLVED
    Reviewer - 0% reviewed      
    vduong  (deleted user)
    Reviewer - 0% reviewed      
    msuleiman  (deleted user)
    Reviewer - 0% reviewed      
    Total   17h 36m 244  
    #permalink

    Objectives

    There are no specific objectives for this review.

    Branches in review

    #permalink

    Issues Raised From Comments

    Key Summary State Assignee
    #permalink

    General Comments

    Behrouz NematiPour

    It seems Jira is doing its best to confuse the user with this whole slider of...

    It seems Jira is doing its best to confuse the user with this whole slider of commits and not showing the comments in order and sometimes even hiding comments.
    I totally don't understand the purpose of these confusing features.
    Source versioning is so simple and don't need to be complicated by this unnecessary features.

    So please forgive me if because of this <not a good adjective> features I put some comments which are irrelevant.
    Thanks,

    plucia  (deleted user)

    No problem Behrouz. Your comment is noted. Since there is no action item in ...

    No problem Behrouz. Your comment is noted.

    Since there is no action item in this comment could you resolve it?

    /resources/images/alarm.svg Added 10
    /resources/images/bell-off.svg Added 11
    /resources/images/bell.svg Added 8
    /scripts/run.sh Changed
    /sources/canbus/messagedispatcher.cpp Changed
    /sources/canbus/messagedispatcher.h Changed 3
    /sources/canbus/messageglobals.h Changed 3
    /sources/canbus/messageinterpreter.cpp Changed 12
    /sources/gui/.../components/ModalDialog.qml Changed 3
    /sources/.../components/NotificationBar.qml Changed 39
    /sources/.../components/NotificationBar2.qml Deleted
    Open in IDE #permalink
    /sources/.../components/NotificationBarSmall.qml Added 6
    Open in IDE #permalink
    /sources/gui/.../components/SettingsItem.qml Added 5
    /sources/gui/qml/components/TouchArea.qml Changed 5
    /sources/gui/qml/components/TouchRect.qml Changed
    Open in IDE #permalink
    /sources/gui/qml/dialogs/Alarm.qml Deleted 6
    Open in IDE #permalink
    /sources/.../dialogs/NotificationDialog.qml Added 13
    /sources/gui/qml/globals/Colors.qml Changed 8
    /sources/gui/qml/globals/Fonts.qml Changed 4
    /sources/gui/qml/globals/Variables.qml Changed
    /sources/.../adjustments/TreatmentAdjustmentBase.qml Changed 7
    /sources/gui/qml/pages/Diagnostics.qml Added 14
    /sources/gui/qml/pages/SettingsHome.qml Changed 9
    /sources/gui/qml/main.qml Changed 3
    /sources/gui/guicontroller.cpp Changed
    /sources/gui/guicontroller.h Changed 3
    /sources/gui/guiglobals.h Changed 5
    /sources/gui/guiview.h Changed 3
    /sources/model/malarmstatus.cpp Changed 16
    /sources/model/malarmstatus.h Changed
    /sources/view/valarmstatus.cpp Changed 8
    /sources/view/valarmstatus.h Changed 3
    /sources/applicationcontroller.cpp Changed
    /sources/applicationcontroller.h Changed 14
    /sources/main.h Changed
    /unittests/tst_messaging.cpp Changed 6
    /unittests/tst_messaging.h Changed
    /unittests/tst_models.cpp Changed 3
    /unittests/tst_models.h Changed 3
    /.gitignore Changed
    /denali.pro Changed
    /denali.pro.user Changed
    Open in IDE #permalink
    /denali.qrc Changed
    /main.cpp Changed 8

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time