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

Integration with xbdeals website #3

Merged
merged 2 commits into from Jan 14, 2022
Merged

Conversation

matk04
Copy link
Contributor

@matk04 matk04 commented Jan 11, 2022

Buenas. Está muy buena la extensión.
Agregué la integración con la web de xbdeals que personalmente la uso mucho.
Hay tres formas diferentes en las que se muestran los precios:

  1. En la home: hay que diferenciar si hay una oferta, un descuento por suscripción, o un precio normal.
  2. En un listado (ya sea abriendo una de las filas de la home o al ingresar una búsqueda): en este caso hay una tag oculta que tiene el precio actual.
  3. En la página de un juego en particular: se lee el juego igual que en la home, solo que tuve que agregar el badge afuera del contenedor de precios porque se rompía el layout de ellos.

De por sí, el tamaño del texto de la extensión queda grande para esta web. Pero podría ser una mejora futura, poder personalizar ese tamaño dependiendo de la web.

@lucasromerodb lucasromerodb self-assigned this Jan 14, 2022
@lucasromerodb lucasromerodb added the enhancement New feature or request label Jan 14, 2022
@lucasromerodb lucasromerodb merged commit a388902 into lucasromerodb:main Jan 14, 2022
@lucasromerodb
Copy link
Owner

lucasromerodb commented Jan 14, 2022

Gracias por el aporte @matk04 !!
Con respecto al font-size del badge de precio: por ahora el foco es soportar todo lo que se pueda (menos steam). Está planeado para luego que el texto se integre mucho mejor a cada web reemplazando el precio original.

Edit: ya lo envié a la Chrome Store para que lo revisen, mañana ya debería estar la versión 1.4 disponible

@matk04 matk04 deleted the xbdeals branch January 14, 2022 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants