-
Notifications
You must be signed in to change notification settings - Fork 511
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
Fix part of #40 & #42: Generalisation mobile devices - Content View - Portrait #1527
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.
LGTM but there are some suggestions
app/src/main/java/org/oppia/app/databinding/StateAssemblerMarginBindingAdapters.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/oppia/app/databinding/StateAssemblerMarginBindingAdapters.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/oppia/app/databinding/StateAssemblerMarginBindingAdapters.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/oppia/app/databinding/StateAssemblerMarginBindingAdapters.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/oppia/app/databinding/StateAssemblerPaddingBindingAdapters.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/oppia/app/databinding/StateAssemblerPaddingBindingAdapters.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/oppia/app/databinding/StateAssemblerPaddingBindingAdapters.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/oppia/app/databinding/StateAssemblerPaddingBindingAdapters.kt
Outdated
Show resolved
Hide resolved
The margins in this reference is different from the mocks: https://xd.adobe.com/view/e8aa4198-3940-47f9-514a-f41cc54457f6-9e9b/screen/9ea1d153-06e3-434f-9bf7-d582d48583b1/specs/ |
Actually, the mocks are inconsistent, that's why we have received this from Chantel, in an email thread. |
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.
LGTM
Explanation
This PR finishes the highfi work of
ContentView-Portrait Mode
for mobile devices only.Note, the strategy that I have followed for margins is that we we introduce only top-start-end margins and bottom margin won't be introduced (as bottom margin for item1 can be treated as top margin for item-2).
Note: When you run the app, it might seem that the text size is small, so to fix that, go to
Options -> StoryTextSize
and set it to medium.Reference:
Screenshots
Checklist