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

nuevo diseño para loginPage #20

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

nuevo diseño para loginPage #20

wants to merge 1 commit into from

Conversation

Feitan173
Copy link
Collaborator

Boton de login animado placeholders cambiados y focus de los inputs cambiados
image

@Feitan173 Feitan173 requested a review from KaruzG October 17, 2023 17:37
color: #f0f0f0;
background-color: rgb(0, 162, 255);
background-color: rgb(0, 162, 255)!important;
Copy link
Owner

Choose a reason for hiding this comment

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

No se recomienda usar !importan

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

no se ponia en azul :(

Copy link
Owner

Choose a reason for hiding this comment

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

A lo mejor es por el reset.css

box-shadow:0 19px 38px rgba(0, 0, 0, 0.3), 0 15px 12px rgba(0, 0, 0, 0.22);
}
form button:hover{
box-shadow:0 10px 20px rgba(0, 0, 0, 0.19), 0 6px 6px rgba(0, 0, 0, 0.23);
Copy link
Owner

Choose a reason for hiding this comment

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

El shadow es demasiado heavy y grande, sería mejor que fuese más sutil.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

concuerdo

Copy link
Owner

Choose a reason for hiding this comment

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

Yo suelo poner un shadow de 0px 1px 5px black

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

pero si pongo ese luego en el transform para que se haga pequeño el boton no queda bien pondre uno un poquito mas grande que ese pero mas pequeño que el original

@@ -37,6 +37,7 @@
<!-- ------------FOOTER------------ -->
<footer>
<p>&copy; 2023 <a href="https://github.com/KaruzG">KaruzG</a>. All rights reserved.</p>
<p><a href="https://github.com/feitan173">Feitan173</a>coxinillo</p>
Copy link
Owner

Choose a reason for hiding this comment

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

Mejor que esto esté a la izq o derecha del de arriba, para no hacer el footer de 2 lineas

@@ -37,6 +37,7 @@
<!-- ------------FOOTER------------ -->
<footer>
<p>&copy; 2023 <a href="https://github.com/KaruzG">KaruzG</a>. All rights reserved.</p>
Copy link
Owner

Choose a reason for hiding this comment

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

div - flex row
flex left flex right

margin: 0;
padding: 0;
}
footer p:nth-child(2){
Copy link
Owner

Choose a reason for hiding this comment

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

Con flex

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.

2 participants