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

feat(electron): add version number to splash screen #590

Merged
merged 3 commits into from
Aug 19, 2020

Conversation

saihaj
Copy link
Member

@saihaj saihaj commented Jul 14, 2020

Summary of PR

Add version number to splash screen

Before
image

After
image

Time spent on PR

40 minutes

Linked issues

Fixes #490

Reviewers

@Harjot1Singh @bhajneet

@bhajneet
Copy link
Member

Does the ellipsis animation still work?

@saihaj
Copy link
Member Author

saihaj commented Jul 14, 2020

Yes

@saihaj
Copy link
Member Author

saihaj commented Jul 14, 2020

Kapture 2020-07-14 at 14 16 22

@saihaj saihaj changed the title feat(electron): add version number and to splash screen feat(electron): add version number to splash screen Jul 19, 2020
@bhajneet
Copy link
Member

UI wise looks fine to me. I will leave code review to @Harjot1Singh

@bhajneet bhajneet removed their request for review July 22, 2020 14:51
Copy link
Member

@Harjot1Singh Harjot1Singh left a comment

Choose a reason for hiding this comment

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

Good solution

app/electron/electron-wrapper.js Outdated Show resolved Hide resolved
app/electron/splashscreen/index.html Outdated Show resolved Hide resolved
app/electron/splashscreen/index.html Outdated Show resolved Hide resolved
@saihaj saihaj requested a review from Harjot1Singh July 22, 2020 19:10
@Harjot1Singh Harjot1Singh merged commit 9c05938 into shabados:dev Aug 19, 2020
@saihaj saihaj deleted the saihaj/issue490 branch August 19, 2020 23:41
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 version number and potentially other data to splash screen
3 participants