In fact, everything that is of class "inline*" overlaps the footnotes.

to prevent the situation, add clear: both to "footnotes" class.
patch for footnotes.css attached to bug report.

CommentFileSizeAuthor
footnotes-6.x-2.0.patch22 byteslelutin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hingo’s picture

Status: Active » Fixed

Good catch, thanks.

Applied and committed, also did the same fix for "footnotes-alternative_layout.css". This fix is available in footnotes-6.x-2.1 and footnotes-5.x-2.1.

PS: Some technicalities...
- in creating patch you've had origfile and newfile the wrong way (patch command catches this though, no problems)
- use two spaces instead of tab
- in drupal, it is prefferred to us -Up as arguments to diff. (Unified diff style)
- even so, it was really nice to have someone posting a patch along with a bug report. It was a small thing and a small bug even, but even so, it made me feel nice. thanks!

lelutin’s picture

- in creating patch you've had origfile and newfile the wrong way (patch command catches this though, no problems)
- in drupal, it is prefferred to us -Up as arguments to diff. (Unified diff style)

I saw those two problems right after committing, oops :\
didn't know about the -p option. it's pretty handy.
This one was my first bug report / patch submittal both to drupal and to open source projects alltogether, the feedback is much appreciated :)

- use two spaces instead of tab

heh, good idea. fixed my vimrc

hingo’s picture

Drupal is my first significant Open Source participation too, I have to say it is a relatively good place to start. Welcome to the club!

Anonymous’s picture

Status: Fixed » Closed (fixed)

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