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

Empty state illustartions #303

Conversation

Darshpreet2000
Copy link
Contributor

Fixes #269

Added Empty State Illustrations for existing pages
(My Profile Page) Contribution Tab
(My Profile Page) Course Tab:
(My Profile Page) Recent Uploads Tab

Screenshots

Before:
Screenshot_2020-01-09-21-01-12-98_0670a9c18916db5d58d88e8d6ddc87a4

After:

Screenshot_2020-01-09-20-59-00-00_0670a9c18916db5d58d88e8d6ddc87a4
Screenshot_2020-01-09-20-59-04-96_0670a9c18916db5d58d88e8d6ddc87a4
Screenshot_2020-01-09-20-59-08-80_0670a9c18916db5d58d88e8d6ddc87a4
Screenshot_2020-01-09-21-01-39-20_0670a9c18916db5d58d88e8d6ddc87a4

Other pages mentioned in issue had already the empty illustrations

@ujjwalagrawal17
Copy link
Member

Those are texts, if possible we will like illustrations for empty state in place of these texts.

@Darshpreet2000
Copy link
Contributor Author

Do you have those Illustrations?
Do you mean an image to be displayed?

@ujjwalagrawal17
Copy link
Member

ujjwalagrawal17 commented Jan 9, 2020

Yes. We need images for that. I don't have the images. We need to search/design them.

@Darshpreet2000
Copy link
Contributor Author

Please send if you have that
I will add them

@Darshpreet2000
Copy link
Contributor Author

Till then Please merge these changes

Copy link
Collaborator

@victorvicari victorvicari left a comment

Choose a reason for hiding this comment

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

Please, take a look at @ujjwalagrawal17 point.

@Darshpreet2000
Copy link
Contributor Author

But Its working correctly

@Darshpreet2000
Copy link
Contributor Author

On other fragments also there is same type of null check using run
On Contribution fragment

@ujjwalagrawal17
Copy link
Member

Okay, I will check and get back.

@ujjwalagrawal17 ujjwalagrawal17 merged commit 39d1766 into WikiEducationFoundation:master Jan 28, 2020
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.

Add Empty State Illustrations for existing pages.
3 participants