Skip to content
This repository has been archived by the owner on Jun 2, 2019. It is now read-only.

Add BrowserErrorView #434

Merged
merged 3 commits into from Mar 10, 2018
Merged

Add BrowserErrorView #434

merged 3 commits into from Mar 10, 2018

Conversation

hewigovens
Copy link
Contributor

Fix #363, It's actually a overlay view

@BlackDuckCoPilot
Copy link

BlackDuckCoPilot commented Mar 8, 2018

Black Duck Security Report

Merging #434 into master will not change security risk.

Click here to see full report

@vikmeup
Copy link
Contributor

vikmeup commented Mar 8, 2018

@hewigovens if we are adding ErrorView, can we have reload/refresh button then?

can you share a screenshot/gif? curious to see how that looks. I would love to have same functionality as Safari has tho.

@hewigovens
Copy link
Contributor Author

@vikmeup Yes, we can add buttons, I will send you the demo via telegram

@vikmeup
Copy link
Contributor

vikmeup commented Mar 9, 2018

@hewigovens let's add reload button Button(style: clear) under error message and center it vertically and should be good to go!

@vikmeup
Copy link
Contributor

vikmeup commented Mar 10, 2018

Looking good @hewigovens !

@vikmeup vikmeup merged commit f813061 into master Mar 10, 2018
@vikmeup vikmeup deleted the issues/363 branch March 10, 2018 00:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants