-
Notifications
You must be signed in to change notification settings - Fork 4
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
Refactor progressbar #369
Refactor progressbar #369
Conversation
@Pe5h4 can you review pls? This MR comes with changes to public/body.css in all applications http://gitlab.teskalabs.int/seacat/seacat-webui/-/merge_requests/139 I will make these changes in other applications progress-v.mp4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One comment, otherwise cool!
src/containers/SplashScreen.js
Outdated
style={{maxWidth: "38%"}} | ||
/> : <h1>Loading ...</h1>} | ||
<div className="progress mt-3" style={{maxWidth: "38%", margin: "20px auto 0 auto"}}> | ||
style={{width: "38%", position: "absolute", bottom: '60%', left: '50%', transform: 'translate(-50%, 0)'}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Aringoaway Is it possible to add those style to splashscreen.scss (or some other general stylesheet e.g. styles/index.scss
) ? Or does it have to be styled directly in the component due to pending of the scss files on application initialization?
Instead of Spinner add SplashScreen