-
Notifications
You must be signed in to change notification settings - Fork 13
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
816-NEW-dashboard-add-text #855
Conversation
Remove broken link to dashboard screenshot. Placeholder for a new one.
Removed broken link old dashboard screenshot. Placeholder for a new one.
making linter happy
linting
Explain what the dashboard is. Use a smaller logo on landing page.
shorter lines to make linter happy
@elboyran I like the text for the home page! I left a few small comments on the text. I also fixed the dashboard tests, they all pass now |
Co-authored-by: Laura <95367983+laurasootes@users.noreply.github.com>
Co-authored-by: Laura <95367983+laurasootes@users.noreply.github.com>
fixing typo Co-authored-by: Laura <95367983+laurasootes@users.noreply.github.com>
fixing typo Co-authored-by: Laura <95367983+laurasootes@users.noreply.github.com>
fixing typo Co-authored-by: Laura <95367983+laurasootes@users.noreply.github.com>
@cwmeijer did you find out why your PRs notebook checks are failing? |
The macos-related checks are failing, not related to this PR. |
@elboyran I added the collapsable section, do you like it this way? |
Hi @laurasootes , I like it a lot.
|
|
|
@elboyran I moved up the texted and removed the centering of the image. |
separating line between description explanation and the example text for images
separating line between description explanation and the example text Tabular
separating line between description explanation and the example text: Text
separating line between description explanation and the example text: Tabular classification
separating line between description explanation and the example text: Time series
Issue #816