Currently the module adds a clearing div using JS in a behavior. This caused a bug where other caused lots of those divs being added.

Besides that I don't see it's purpose.

CommentFileSizeAuthor
#5 3059216-3-remove-clearing-div.patch1.02 KBpurvitagupta
#2 3059216-2-remove-clearing-div.patch400 bytesluksak

Comments

Lukas von Blarer created an issue. See original summary.

luksak’s picture

luksak
he/him
commented
Status: Active » Needs review
StatusFileSize
new 3059216-2-remove-clearing-div.patch400 bytes

Status: Needs review » Needs work

The last submitted patch, 2: 3059216-2-remove-clearing-div.patch, failed testing. View results

kala4ek’s picture

kala4ek
Primary language Russian
Location 🇷🇺 Novosibirsk
commented
Assigned: Unassigned » kala4ek
purvitagupta’s picture

purvitagupta commented
StatusFileSize
new 3059216-3-remove-clearing-div.patch1.02 KB

Remove clearing div With clearfix class. please review it .

purvitagupta’s picture

purvitagupta commented
Status: Needs work » Needs review

  • kala4ek committed 9268bfd on 8.x-2.x 
    Issue #3059216 by Lukas von Blarer, purvitagupta, kala4ek: Remove...
kala4ek’s picture

kala4ek
Primary language Russian
Location 🇷🇺 Novosibirsk
commented
Status: Needs review » Fixed

Committed to dev release.

Status: Fixed » Closed (fixed)

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