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

some Optimizations for tablets #647

Merged
merged 4 commits into from
May 16, 2018
Merged

some Optimizations for tablets #647

merged 4 commits into from
May 16, 2018

Conversation

connyduck
Copy link
Collaborator

resolves #421
Step one: Show ComposeActivity as a dialog on tablets!

vs.

@connyduck connyduck added this to the Tusky 1.9 milestone May 10, 2018
@connyduck
Copy link
Collaborator Author

Step two: limit timeline width on larger devices. Onfurtunately, I found no way of doing this without an additional FrameLayout.

vs.

@trwnh
Copy link

trwnh commented May 10, 2018

Rather than limiting timeline width, perhaps try for a multi-column approach? If not that, then maybe figure out a better use for the whitespace around the edges (putting notifications on the side of the timelines?)

@connyduck
Copy link
Collaborator Author

I will only do stuff that can be achieved with layout/style changes because I do not want to add additional code complexity or spend much time on this. Yes, multi column would be awesome, but its too big of a change for now.

@connyduck
Copy link
Collaborator Author

Ok so I think I got the most important stuff. I will fix the AboutActivity once #646 is done.

Copy link
Collaborator

@charlag charlag left a comment

Choose a reason for hiding this comment

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

LGTM! I don't have a tablet to check it but code looks good!

<?xml version="1.0" encoding="utf-8"?>
<shape xmlns:android="http://schemas.android.com/apk/res/android">
<corners android:radius="5dp" />
<solid android:color="@color/window_background_dark" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

  1. Maybe pull out radius to dimen?
  2. Maybe you could create one drawable but color resource would be actually different?

@@ -0,0 +1,20 @@
<?xml version="1.0" encoding="utf-8"?>
Copy link
Collaborator

Choose a reason for hiding this comment

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

I was surprised that when it's not ConstraintLayout there's no maxWidth

@connyduck connyduck merged commit d4572a9 into master May 16, 2018
@connyduck connyduck deleted the tablet-optimization branch May 16, 2018 17:14
nailyk-fr pushed a commit to nailyk-fr/Tusky that referenced this pull request Aug 4, 2018
* show ComposeActivity as Dialog on tablets

* limit timeline width on large devices

* limit width on large devices for thread view and search

* increase media preview size on large devices
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tablet optimization
3 participants