Skip to content

Conversation

@Rulino
Copy link
Contributor

@Rulino Rulino commented Oct 17, 2018

Title (required)

Fixes #1912 Empty screen if no upload

Description (required)

Fixes #1912 Empty screen if no upload

Added text view for empty page showing welcome text in case there are no uploads. Once there are files the message is hid.

Tests performed (required)

Tested on API 25 & Galaxy J5, with {BetaDebug}

Screenshots showing what changed (optional)

Without and with uploads:

screenshot_20181017-123328
screenshot_20181017-123511

@codecov-io
Copy link

codecov-io commented Oct 17, 2018

Codecov Report

Merging #1940 into master will decrease coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #1940      +/-   ##
=========================================
- Coverage    4.55%   4.54%   -0.01%     
=========================================
  Files         199     199              
  Lines       10108   10113       +5     
  Branches      899     900       +1     
=========================================
  Hits          460     460              
- Misses       9613    9618       +5     
  Partials       35      35
Impacted Files Coverage Δ
...mmons/contributions/ContributionsListFragment.java 0% <0%> (ø) ⬆️
...w/commons/contributions/ContributionsActivity.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 8526605...7ff4c91. Read the comment docs.

android:layout_height="wrap_content"
android:text="@string/no_uploads"
android:gravity="center"
android:layout_centerInParent="true"
Copy link
Contributor

Choose a reason for hiding this comment

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

Add left and right margins please :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done
screenshot_20181019-111508

@nicolas-raoul
Copy link
Member

Looks good to me.
Styling can be improved later.

@maskaravivek maskaravivek merged commit 2af57fc into commons-app:master Oct 20, 2018
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.

4 participants