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

fix: back to SEI link on login page #607

Merged
merged 5 commits into from
Feb 27, 2024

Conversation

ShashwatPS
Copy link
Contributor

@ShashwatPS ShashwatPS commented Dec 18, 2023

Fixed the Back to SEI Website Link on the login page and also provided some padding on the left as it was overlapping with the image.

Fixes #606

Copy link

netlify bot commented Dec 18, 2023

👷 Deploy request for seium-stg pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 0c2ccd5

@joaodiaslobo joaodiaslobo changed the title fix: Fixed back to SEI Website link on the login page fix: back to SEI link on login page Dec 19, 2023
@@ -16,12 +16,12 @@ import { LoginForm } from "./components";
function Login() {
return (
<div className="min-h-screen select-none overflow-hidden bg-secondary">
<Return componentStyle="sm:ml-20 mt-20 sm:mt-20" />
<Return componentStyle="sm:ml-20 mt-20 sm:mt-20 z-10 pl-10" />
Copy link
Member

Choose a reason for hiding this comment

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

Thank you 🙇!
Just make sure to remove the left padding when on mobile screens to center the text properly, like this:

Suggested change
<Return componentStyle="sm:ml-20 mt-20 sm:mt-20 z-10 pl-10" />
<Return componentStyle="sm:ml-20 mt-20 sm:mt-20 z-10 sm:pl-10" />

layout/Login/Login.tsx Outdated Show resolved Hide resolved
Copy link

netlify bot commented Feb 11, 2024

👷 Deploy request for lazuli-stg pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit faa7e2b

Co-authored-by: tiago-bacelar <70779845+tiago-bacelar@users.noreply.github.com>
@ruilopesm ruilopesm dismissed stale reviews from tiago-bacelar and joaodiaslobo February 27, 2024 22:19

Outdated

@ruilopesm ruilopesm merged commit 911a04d into cesium:main Feb 27, 2024
1 check passed
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.

Back to SEI link appearing behind mascot image on certain screen resolutions
4 participants