Skip to content
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

Fixes #2119 #2118 #2134

Merged
merged 3 commits into from
Oct 12, 2020
Merged

Conversation

riturajjain2000
Copy link
Member

@riturajjain2000 riturajjain2000 commented Oct 5, 2020

Fixes #2119 #2118

Changes: Added Tablet layout for multimeter , unlocked landscape mode for some activities and fixed landscape mode of Splash Screen
Had to add some values in dimens.xml as Tablet layout need more space than normal one.

Screenshot/s for the changes:
Android Emulator - Pixel_C_API_29_5556 05-10-2020 20_29_41
Android Emulator - Pixel_C_API_29_5556 05-10-2020 20_28_59
Android Emulator - Pixel_C_API_29_5556 05-10-2020 20_29_34
Android Emulator - Pixel_C_API_29_5556 05-10-2020 21_27_51

Checklist:

  • I have used resources from strings.xml, dimens.xml and colors.xml without hard-coding them
  • No modifications done at the end of resource files strings.xml, dimens.xml or colors.xml
  • I have reformatted code in every file included in this PR [CTRL+ALT+L]
  • My code does not contain any extra lines or extra spaces
  • I have requested reviews from other members

APK for Testing:
tablet_layout-and-landscape_splash_screen.zip
Made_changes asked.zip

Added Tablet layput for multimeter , unlocked landscape mode for some activities and fixed landscape mode of Splash Screen
Added Tablet layput for multimeter , unlocked landscape mode for some activities and fixed landscape mode of Splash Screen
Copy link
Collaborator

@CloudyPadmal CloudyPadmal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The AndroidManifest changes would affect the phone layout. Don't change the orientation there.

@CloudyPadmal CloudyPadmal added the Enhancement Improvement to an existing feature label Oct 7, 2020
android:windowSoftInputMode="stateHidden|adjustResize" />
<activity
android:name=".activity.LuxMeterActivity"
android:screenOrientation="portrait" />
<activity
android:name=".activity.WaveGeneratorActivity"
android:screenOrientation="portrait" />
android:screenOrientation="user" />
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revert back and use a layout file for tabs. Refer to comments for a screenshot.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok sir , will do that

<activity
android:name=".activity.PowerSourceActivity"
android:screenOrientation="portrait"
android:screenOrientation="user"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revert back and use a layout file for tabs. Refer to comments for a screenshot.

@CloudyPadmal
Copy link
Collaborator

Now the Multimeter ,  Wavegenerator and Powersource Activities  will remain only in portrait mode in normal sized phones and will have both portrait and landscape modes in Tablets
Copy link
Collaborator

@CloudyPadmal CloudyPadmal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the PR! 👍

@CloudyPadmal CloudyPadmal merged commit a49e6b7 into fossasia:development Oct 12, 2020
@riturajjain2000
Copy link
Member Author

Thank you for the PR! 👍

Happy to help sir :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Improvement to an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Request: Horizontal layouts for all screens
3 participants