Skip to content

AC-560 Wrap Parent layout to match text size in charts list#640

Merged
f4ww4z merged 1 commit intoopenmrs:masterfrom
VibhorChinda:AC-560
Dec 24, 2019
Merged

AC-560 Wrap Parent layout to match text size in charts list#640
f4ww4z merged 1 commit intoopenmrs:masterfrom
VibhorChinda:AC-560

Conversation

@VibhorChinda
Copy link
Collaborator

Description of what I changed

Wrap Parent layout to match text size in charts list and setting horizontal divider height to match parent.

Issue I worked on

JIRA Issue: https://issues.openmrs.org/browse/AC-560

Checklist: I completed these to help reviewers :)

  • My pull request only contains ONE single commit
    (the number above, next to the 'Commits' tab is 1).
  • I have added tests to cover my changes. (If you refactored
    existing code that was well tested you do not have to add tests)
  • All new and existing tests passed.
  • My pull request is based on the latest changes of the master branch.

@codecov-io
Copy link

codecov-io commented Dec 19, 2019

Codecov Report

Merging #640 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #640   +/-   ##
======================================
  Coverage    14.4%   14.4%           
======================================
  Files         202     202           
  Lines        9223    9223           
  Branches      791     791           
======================================
  Hits         1329    1329           
  Misses       7813    7813           
  Partials       81      81

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 858dce9...88a8371. Read the comment docs.

@VibhorChinda
Copy link
Collaborator Author

@f4ww4z please have a look at the PR

@f4ww4z
Copy link
Collaborator

f4ww4z commented Dec 23, 2019

@VibhorChinda can you post a screenshot here?

@VibhorChinda
Copy link
Collaborator Author

Screenshot 2019-12-24 at 1 19 34 AM

@VibhorChinda
Copy link
Collaborator Author

@f4ww4z added the layout screenshot here please have a look. The vertical view now matches the height of its parent view

@f4ww4z f4ww4z merged commit 8cd532b into openmrs:master Dec 24, 2019
rishabh-997 pushed a commit to rishabh-997/openmrs-contrib-android-client that referenced this pull request Dec 27, 2019
rishabh-997 pushed a commit to rishabh-997/openmrs-contrib-android-client that referenced this pull request Dec 28, 2019
AC-560: Wrap Parent layout to match text size in charts list (openmrs#640)

[AC-638] Added inputType for capitalizing the first letter (openmrs#643)

added first time intro slides to app

completed requested changes

added seven intro slides

[AC-638] Added inputType for capitalizing the first letter (openmrs#643)

resolved conflicts

resolved conflicts with splash screen

squashing commits
@VibhorChinda VibhorChinda deleted the AC-560 branch January 13, 2020 03:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants