In order to improve the UI/theming in commerce_discount, inline_conditions need some love (see caption)

CommentFileSizeAuthor
#7 inline_conditions-improve_ui-2025125-7.patch2.95 KBCookiz
#6 inline_conditions-improve_ui-2025125-6.patch3.72 KBCookiz
#1 inline_conditions-improve_ui-2025125.patch6.48 KBCookiz
Capture d’écran 2013-06-21 à 15.00.15.png75.8 KBCookiz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Cookiz’s picture

Cookiz CreditAttribution: Cookiz commented
FileSize
inline_conditions-improve_ui-2025125.patch6.48 KB
jkuma’s picture

jkuma CreditAttribution: jkuma commented
Status: Active » Needs review
jkuma’s picture

jkuma CreditAttribution: jkuma commented
Status: Needs review » Reviewed & tested by the community

Looks good for me and the patch is working well.

jkuma’s picture

jkuma CreditAttribution: jkuma commented
Status: Reviewed & tested by the community » Fixed

Committed in 7.x-1.x-dev branch.

jkuma’s picture

jkuma CreditAttribution: jkuma commented
Status: Fixed » Needs work

We need to remove css parts which are related to commerce discount.

Cookiz’s picture

Cookiz CreditAttribution: Cookiz commented
FileSize
inline_conditions-improve_ui-2025125-6.patch3.72 KB

Reworked version.

Cookiz’s picture

Cookiz CreditAttribution: Cookiz commented
FileSize
inline_conditions-improve_ui-2025125-7.patch2.95 KB

Oops, I've added code for RTL support inside the previous patch... please use this one instead.

jkuma’s picture

jkuma CreditAttribution: jkuma commented
Status: Needs work » Reviewed & tested by the community

Working well and looks good to me.

jkuma’s picture

jkuma CreditAttribution: jkuma commented
Status: Reviewed & tested by the community » Fixed

Committed in 7.x-1.x-dev branch.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.