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

overhauled about-fragment to make it less cramped #760

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

newhinton
Copy link
Contributor

No description provided.

@jspricke
Copy link
Member

Can you improve the horizontal layout? This looks broken with your patch:
Screenshot_20200821-155552_Etar

@Gitsaibot
Copy link
Contributor

I haven't tested this yet, but now we have 200 lines more code and two files instead of one.

@newhinton
Copy link
Contributor Author

yup, that is expected. I created a landscape version of the about xml, because just stacking everything on top of each other does not look good. So i basically created two adjacent "stacks" that display the information, and therefore a new file was required

@Gitsaibot
Copy link
Contributor

Can you provide screenshots before/after and resolve string.xml conflict please.

@newhinton
Copy link
Contributor Author

Screenshot_1643192192
Screenshot_1643192323

@Gitsaibot i have added screenshots and rebased my branch. The huge statusbar is an artifact of the emulator

@Gitsaibot
Copy link
Contributor

Gitsaibot commented Jan 29, 2022

This is not rebased. Your PR has 284 commits! I can't check it in this state.

@newhinton newhinton force-pushed the feature/noid/about_overhaul branch 2 times, most recently from 1abf3c4 to 2303365 Compare February 2, 2022 13:08
# Conflicts:
#	res/layout/about.xml
#	res/values/strings.xml
@newhinton
Copy link
Contributor Author

@Gitsaibot This should resolve the conflicts

@Gitsaibot
Copy link
Contributor

Icons white, copyright text at different position (portrait/landscape), text spacing different
Tablet_hoch
Tablet

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