Limit image size breaking retina is confusing
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Comienza reproduciendo la interacción descrita en el issue: activa Retina, luego activa Limit image size y observa el comportamiento actual de la configuración. El cambio estará terminado cuando se informe a los usuarios de que las configuraciones no se pueden activar juntas y se desactive la configuración en conflicto como se propone.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
What problem does this address?
I couldn't understand why images on Codeinwp are not looking great and I have realised this is because retina is disabled. It was hard to detect because in fact, the setting was enabled, just that I noticed after in the fine print of limit image size, which is enabled by default now, that Retina doesn't work with that.
If that setting is coming on by default, I don't think anyone would read through the whole paragraph to see that retina images aren't working, considering they can just enable that before.
What is your proposed solution?
If we want to leave the settings by default as they are now, I suggest that when a user tries to enable retina we alert them that we disabled limit image size setting for that to work.
Also if the image size is disabled and the user enables it while retina is also active, we alert them that we need to disable retina, since those can't function together.
Basically don't let the user believe that they can enable both of them at the same time.
Will this feature require documentation? (Optional)
None
- Lenguaje dominante
- PHP
- Estrellas
- 73
- Forks
- 14
- Merge medio
- 2 d 12 h
- PR fusionados (30 d)
- 18
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de Codeinwp/optimole-wp
-
bug-report bug-report-triage crash-report regression
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
Codeinwp/optimole-wp#1163 ·
Los mantenedores suelen responder en 1 día
-
bug-report bug-report-triage crash-report
Dificultad 4/5 3-5 días Aptitud para principiantes 65/100
Codeinwp/optimole-wp#1162 ·
Los mantenedores suelen responder en 1 día
-
customer report feature-request-triage
Dificultad 3/5 1-2 días Aptitud para principiantes 65/100
Codeinwp/optimole-wp#1161 ·
Los mantenedores suelen responder en 1 día
-
bug-report bug-report-triage customer report regression
Dificultad 4/5 3-5 días Aptitud para principiantes 56/100
Codeinwp/optimole-wp#1159 ·
Los mantenedores suelen responder en 1 día
-
customer report feature-request-triage
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
Codeinwp/optimole-wp#1151 ·
Los mantenedores suelen responder en 1 día
Todos los issues de Codeinwp/optimole-wp
Issues similares
-
UX
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
ProfessionalWiki/NeoWiki#1573 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
-
sync-en
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día
-
sync-en
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día