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

Restyling Payment methods overview #6

Closed
wants to merge 2 commits into from
Closed

Restyling Payment methods overview #6

wants to merge 2 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Apr 22, 2015

No description provided.

@lvgunst
Copy link
Contributor

lvgunst commented Apr 22, 2015

💥 Wanneer je bij de instelling "Bankenlijst" kiest voor "Altijd zichtbaar" of "Zichtbaar na klikken betaalmethode" wordt het blok van iDEAL niet mooi weergegeven. Die verspringt omdat de bankenlijst nu zichtbaar is. Volgens mij is dit niet makkelijk op te lossen. Ik vraag me dus af of we dit moeten introduceren in onze module. Dit gaat denk ik alleen maar meer vragen opleveren. @adriaanmol

1.5
screen shot 2015-04-22 at 14 54 57

1.6
screen shot 2015-04-22 at 14 48 41

@ghost
Copy link
Author

ghost commented Apr 22, 2015

Er is wel een manier om dat voor elkaar te krijgen.

Dit zou het resultaat zijn.

(op het kleinst).

Dit lukt met een absolute position met een relative position als parent.

Het lukt niet echt om het dan ook automatisch te laten resizen naar rechts naarmate het breder wordt.

Dan zou dit bovenstaand de oplossing zijn. Waarbij het Bankenlijstje dus dicht tegen de text iDeal aan komt.

Ik hoor het graag of dat acceptabel is, dan zal ik het doorvoeren.

From: Lennard van Gunst [mailto:notifications@github.com]
Sent: woensdag 22 april 2015 15:00
To: mollie/Prestashop
Cc: George van Engers
Subject: Re: [Prestashop] Restyling Payment methods overview (#6)

https://assets-cdn.github.com/images/icons/emoji/unicode/1f4a5.png Wanneer je bij de instelling "Bankenlijst" kiest voor "Altijd zichtbaar" of "Zichtbaar na klikken betaalmethode" wordt het blok van iDEAL niet mooi weergegeven. Die verdringt omdat de bankenlijst nu zichtbaar is. Volgens mij is dit niet makkelijk op te lossen. Ik vraag me dus af of we dit moeten introduceren in onze module. Dit gaat denk ik alleen maar meer vragen opleveren. @adriaanmol https://github.com/adriaanmol

1.5
https://cloud.githubusercontent.com/assets/130915/7274731/9fe9eca6-e8ff-11e4-959c-793b966fe24e.png

1.6
https://cloud.githubusercontent.com/assets/130915/7274651/ec6c9c46-e8fe-11e4-844b-1ddd4dcff8f3.png


Reply to this email directly or view it on GitHub #6 (comment) . https://github.com/notifications/beacon/ADrkEzD1CgkzjYHRgRnfIK84MupHIb6Dks5oB5LUgaJpZM4EGA7X.gif

@lvgunst
Copy link
Contributor

lvgunst commented Apr 22, 2015

George, als je reageert via mail lijkt hij geen afbeeldingen mee te sturen. NVM ik zie de afbeeldingen wel in de mailconversatie.

Werken met absolute positionering lijkt me geen goed idee, omdat dit denk ik niet werkt in elk thema.

@lvgunst
Copy link
Contributor

lvgunst commented Apr 23, 2015

Na overleg besloten om dit niet op te nemen in de module. Styles moeten veranderd worden in het PrestaShop thema.

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