fix(new-plan-selection): ajustes UI, mensajes de plan actual y consistencia visual#2883
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Resumen
Este PR aplica ajustes funcionales y visuales en
NewPlanSelectionpara mejorar consistencia de UX en Contactos/Envíos, estados comerciales y espaciados de layout.Contexto
Durante QA de
NewPlanSelectionse detectaron varios puntos de consistencia visual y de reglas de interacción:Cambios implementados
1) Contactos: ocultar controles en escenarios no comprables
En
ContactsPlanahora se ocultan Suscripción y Promocode cuando:Más de 100.0002) Mensaje de plan actual
Se incorporó mensaje de plan actual en formato consistente:
No puedes seleccionar este Plan porque es el que posees actualmente.byContactbyEmailPromocode3) Envíos: consistencia de cartel informativo
En el mensaje de
Más de 100k Contactos:CONTÁCTANOSse ubica debajo del texto (no a la derecha)4) Breakdown de prepayment (trimestral/semestral/anual)
Se mantiene el salto de línea en el texto de ahorro para mostrar:
1 pago trimestral/semestral/anual de ...Además se agregó test de cobertura para verificar consistencia de los 3 casos.
5) Ajustes de espaciado/layout
Archivos modificados
src/components/BuyProcess/NewPlanSelection/ContactsPlan/index.jssrc/components/BuyProcess/NewPlanSelection/EmailsPlan/index.jssrc/components/BuyProcess/NewPlanSelection/index.styles.jssrc/components/BuyProcess/NewPlanSelection/index.test.jssrc/i18n/es.jssrc/i18n/en.jsValidaciones ejecutadas
./node_modules/.bin/prettier --checksobre archivos modificadosyarn test:related --runInBand src/components/BuyProcess/NewPlanSelection/ContactsPlan/index.js src/components/BuyProcess/NewPlanSelection/EmailsPlan/index.js src/components/BuyProcess/NewPlanSelection/index.styles.js src/components/BuyProcess/NewPlanSelection/index.test.js src/i18n/es.js src/i18n/en.jsResultado:
181suites OK688tests OK (3skipped)Referencias visuales
Se aplicaron cambios basados en las capturas compartidas durante esta iteración:
CONTÁCTANOSdebajo del textoNota: las imágenes adjuntas en el chat no tienen URL pública directa para incrustarlas automáticamente en GitHub. Si las subes al PR (o a
docs/assets) las puedo insertar en esta descripción con markdown.