issue #8, reducir altura cover img en móvil #12

2021-10-03T16:41:05+02:00

pyrenaicus commented

2021-10-03 16:41:05 +02:00

(Migrated from github.com)

No description provided.

tanrax commented

2021-10-03 23:07:18 +02:00

(Migrated from github.com)

@pyrenaicus Varias cosas.

No veo que el cambio del CSS cambie el diseño de móvil, ni con tu rama ni yo imitando el commit que has realizado. ¿Podrías revisarlo y/o dejar una captura de como lo ves?
Muy mala práctica añadir una medida suelta en cualquier código. Para eso existen las variables.
Revisa como se indica la medida de la imagen en Desktop. Añade tu propia variable de altura dentro de mobile.css, arriba imitando como está main.css
Me acabo de dar cuenta que eres el mismo autor del pull request con el tema oscuro. ¿Podrías hacer una limpieza del código CSS? He revisado las variable y no usas todas las de nord.css. ¿Para que están?. Podrías borrar las sobrantes y las que utilices darles un nombre semántico.

--color__nord--dark:
--color__nord--light
--color__nord--medium:

nord8 no significa nada para un diseñador web..
Etc. Incluso podrías directamente añadirlo al main del tema.

Ánimo!

@pyrenaicus Varias cosas. 1. No veo que el cambio del CSS cambie el diseño de móvil, ni con tu rama ni yo imitando el commit que has realizado. ¿Podrías revisarlo y/o dejar una captura de como lo ves? 2. Muy mala práctica añadir una medida suelta en cualquier código. Para eso existen las variables. Revisa como se indica la medida de la imagen en Desktop. Añade tu propia variable de altura dentro de mobile.css, arriba imitando como está main.css 3. Me acabo de dar cuenta que eres el mismo autor del pull request con el tema oscuro. ¿Podrías hacer una limpieza del código CSS? He revisado las variable y no usas todas las de nord.css. ¿Para que están?. Podrías borrar las sobrantes y las que utilices darles un nombre semántico. ```css --color__nord--dark: --color__nord--light --color__nord--medium: ``` nord8 no significa nada para un diseñador web.. Etc. Incluso podrías directamente añadirlo al main del tema. Ánimo!

tanrax commented

2021-10-06 08:15:52 +02:00

(Migrated from github.com)

Buen trabajo! Esta tarde la reviso 😄

pyrenaicus commented

2021-10-06 08:20:36 +02:00

(Migrated from github.com)

Hola @tanrax, he realizado los cambios indicados, espero que el cambio en mobile.css se pueda ver, adjunto una captura de como se ve.

😅 Aún me falta pillar como va esto, veo que acabo de hacer un PR, con solo hacer git push desde terminal...

Hola @tanrax, he realizado los cambios indicados, espero que el cambio en mobile.css se pueda ver, adjunto una [captura](https://drive.google.com/file/d/1qLRcme3kd38ydMBgRoxI0QRDO6RfD0kH/view?usp=sharing) de como se ve. 😅 Aún me falta pillar como va esto, veo que acabo de hacer un PR, con solo hacer git push desde terminal...

tanrax commented

2021-10-06 22:13:29 +02:00

(Migrated from github.com)

Vale... ya veo que pasa. Tus estilos se sobrescriben con los de mobile.css. Ya les cambio el orden no te preocupes. Buen trabajo.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: andros/RSSPAPER#12