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

Simplify welcome layouts #2741

Merged
merged 6 commits into from
Apr 16, 2019
Merged

Simplify welcome layouts #2741

merged 6 commits into from
Apr 16, 2019

Conversation

domdomegg
Copy link
Member

@domdomegg domdomegg commented Mar 24, 2019

Description

Works on #2643

What changes did you make and why?

Refactored welcome layouts so they're much more structurally similar, moved repeatedly used bullet to it's own file, removed a load of unused xml properties.

Tests performed

Tested 2.10.1-debug-refactor-welcome-layouts~c8f018810 in portrait mode

Tested 2.10.1-debug-refactor-welcome-layouts~c23e6c82f in portrait and landscape mode

@codecov-io
Copy link

codecov-io commented Mar 24, 2019

Codecov Report

Merging #2741 into master will increase coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #2741      +/-   ##
=========================================
+ Coverage    3.42%   3.43%   +<.01%     
=========================================
  Files         246     246              
  Lines       12193   12182      -11     
  Branches     1071    1071              
=========================================
  Hits          418     418              
+ Misses      11744   11733      -11     
  Partials       31      31
Impacted Files Coverage Δ
...main/java/fr/free/nrw/commons/WelcomeActivity.java 0% <ø> (ø) ⬆️
.../java/fr/free/nrw/commons/WelcomePagerAdapter.java 0% <0%> (ø) ⬆️

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 6bde4f9...4011972. Read the comment docs.

@domdomegg
Copy link
Member Author

Files that might be useful for people in the future looking at this PR, that I compiled from older versions of the app

tulip
selfie_x
rainbow_bridge
proprietary_x
mount_zao
llamas
collage_do_upload
collage_dont_upload

@domdomegg domdomegg changed the title [WIP] Simplify welcome layouts Simplify welcome layouts Mar 27, 2019
@domdomegg
Copy link
Member Author

Tested 2.10.1-debug-refactor-welcome-layouts~c23e6c82f

Welcome activity works

Beta Commons Upload

All automated tests pass

@domdomegg domdomegg changed the title Simplify welcome layouts [WIP] Simplify welcome layouts Mar 28, 2019
@domdomegg domdomegg changed the title [WIP] Simplify welcome layouts Simplify welcome layouts Mar 28, 2019
@neslihanturan neslihanturan merged commit 5d299b1 into commons-app:master Apr 16, 2019
@domdomegg domdomegg deleted the refactor-welcome-layouts branch April 16, 2019 11:18
@domdomegg
Copy link
Member Author

@neslihanturan Thanks for merging!

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