Css properties from the Config Form are not getting applied. It was due to CSS Selector Precedence.

CommentFileSizeAuthor
#9 styling_of_back_to_top-3080772.patch11.42 KBmeena.bisht
#7 configfix.patch5.69 KBsumit-k
#5 Back_To_Top.png642.75 KBabhisekmazumdar
#2 3080772_css_selector.patch461 bytesmeena.bisht
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

MEENA BISHT created an issue. See original summary.

meena.bisht’s picture

meena.bisht CreditAttribution: meena.bisht commented
FileSize
3080772_css_selector.patch461 bytes

Please find the attached patch to fix the issue of CSS properties not getting applied from Config form.

meena.bisht’s picture

meena.bisht CreditAttribution: meena.bisht commented
Status: Active » Needs review
sumit-k’s picture

sumit-k CreditAttribution: sumit-k commented

Applied Patch. It's Working Fine.

abhisekmazumdar’s picture

abhisekmazumdar
he/him
Primary language English
Location India
CreditAttribution: abhisekmazumdar as a volunteer and at QED42 commented
FileSize
Back_To_Top.png642.75 KB

Hi @MEENA BISHT
Thanks for the patch, but I found out that the Border color is not getting applied. Kindly check.

Thank You.

abhisekmazumdar’s picture

abhisekmazumdar
he/him
Primary language English
Location India
CreditAttribution: abhisekmazumdar as a volunteer and at QED42 commented
Status: Needs review » Needs work
sumit-k’s picture

sumit-k CreditAttribution: sumit-k at Axelerant commented
FileSize
configfix.patch5.69 KB

Fixed border configuration issue and coding statndard. Patch Attached. Please Check.

sumit-k’s picture

sumit-k CreditAttribution: sumit-k commented
Status: Needs work » Needs review
meena.bisht’s picture

meena.bisht CreditAttribution: meena.bisht at QED42 commented
FileSize
styling_of_back_to_top-3080772.patch11.42 KB

Following issues have been fixed:

  1. Button placement of was not working as per the configuration.
  2. Added padding and increased border radius.
  3. Hover effect was not working properly.
meena.bisht’s picture

meena.bisht CreditAttribution: meena.bisht at QED42 commented
Assigned: meena.bisht » Unassigned
ambuj_gupta’s picture

ambuj_gupta CreditAttribution: ambuj_gupta at QED42 commented

Tested and verified the patch added by @meena-bisht. It is working fine.

ambuj_gupta’s picture

ambuj_gupta CreditAttribution: ambuj_gupta at QED42 commented
Status: Needs review » Reviewed & tested by the community

  • MEENA BISHT authored 853f87d on 8.x-1.x 
    Issue #3080772 by MEENA BISHT, bsumit5577, abhisekmazumdar, ambuj30oct:...
sumitmadan’s picture

sumitmadan CreditAttribution: sumitmadan at QED42 commented
Status: Reviewed & tested by the community » Fixed

Pushed and fixed to 8.x1.x. Thank you everyone for the contribution.

Status: Fixed » Closed (fixed)

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