Differential D15352

Hide seat reservation section if no seat is reservated
ClosedPublic
Actions

Authored by nicolasfella on Sep 8 2018, 3:39 PM.

Details

Reviewers
vkrause
Commits
R1007:2323e87bcd06: Hide seat reservation section if no seat is reservated
Summary

On a train without seat reservation hide the section instead of displaying empty forms.

Alternative suggestion: Show "No seats are reserved" message

Test Plan

Open trip that contains trains with and without seat reservation. Seats are displayed when
available and hidden when not

Diff Detail

Repository
R1007 Itinerary - Travel Assistance Application
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
nicolasfella created this revision.Sep 8 2018, 3:39 PM
Restricted Application added a project: KDE PIM. · View Herald TranscriptSep 8 2018, 3:39 PM
Restricted Application added a subscriber: kde-pim. · View Herald Transcript
nicolasfella requested review of this revision.Sep 8 2018, 3:39 PM
nicolasfella edited the summary of this revision. (Show Details)Sep 8 2018, 3:39 PM
nicolasfella added a project: KDE Itinerary.
vkrause accepted this revision.Sep 8 2018, 3:43 PM

Thanks!

This revision is now accepted and ready to land.Sep 8 2018, 3:43 PM
Closed by commit R1007:2323e87bcd06: Hide seat reservation section if no seat is reservated (authored by nicolasfella). · Explain WhySep 8 2018, 3:44 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/app/TrainPage.qml (3 lines)

Diff 41207

View Options

src/app/TrainPage.qml

Loading...