lang activity layout
ClosedPublic

Authored by dekumar on Feb 12 2019, 3:41 PM.

Details

Diff Detail

Repository
R2 GCompris
Branch
lang
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 9099
Build 9117: arc lint + arc unit
dekumar created this revision.Feb 12 2019, 3:41 PM
Restricted Application added a project: KDE Edu. · View Herald TranscriptFeb 12 2019, 3:41 PM
Restricted Application added a subscriber: kde-edu. · View Herald Transcript
dekumar requested review of this revision.Feb 12 2019, 3:41 PM
dekumar retitled this revision from Overriding issuesfixed to lang activity layout.Feb 12 2019, 3:44 PM
dekumar added a reviewer: GCompris: Improvements.

Thank you for the patch.
Can you please provide more information on what is fixed? The original issue had 2 issues (with screenshots to better understanding). It would make reviewers gain more time to see the fix and know what has changed without having to apply the patch.

It seems the first point have been fixed (override of the score) but not the second point, is it expected?

@jjazeix The second issue also had been fixed
As the issue 2 , the bar was overriding the last option https://pasteboard.co/HGSCFl7.png
But it has been fixed now ( https://pasteboard.co/I0RdhS6.png )( https://pasteboard.co/I0ReGQ2.png )

dekumar updated this revision to Diff 51824.Feb 16 2019, 5:36 AM
  • Overriding issues fixed

The following overriding issues have been fixed :

1 . Bar overrides the last option. https://pasteboard.co/HGSCFl7.png
After fixing this issue https://ibb.co/XtMNPVJ

  1. In the vertical mode the Ok button is not visible. https://ibb.co/wBmcFLw

After fixing this issue https://ibb.co/71451p5

dekumar updated this revision to Diff 53017.Mar 2 2019, 5:50 PM
  • Typo fixed
jjazeix accepted this revision.Mar 2 2019, 6:16 PM
This revision is now accepted and ready to land.Mar 2 2019, 6:16 PM