Issue Details (XML | Word | Printable)

Key: MDL-18831
Type: Bug Bug
Status: Open Open
Priority: Trivial Trivial
Assignee: moodle.com
Reporter: Ryan Smith
Votes: 0
Watchers: 2
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Moodle

course search box misaligned

Created: 10/Apr/09 08:34 PM   Updated: 16/Feb/10 10:25 PM
Component/s: Course
Affects Version/s: 1.9.4
Fix Version/s: None

File Attachments: 1. Text File MDL-18831.patch (0.9 kB)
2. Text File replacing-fieldset-with-div.patch (0.9 kB)

Image Attachments:

1. search.png
(3 kB)
Environment: Firefox 3.0.8

Participants: Jose Cedeno, Martín Langhoff, moodle.com and Ryan Smith
Security Level: None
Affected Branches: MOODLE_19_STABLE


 Description  « Hide
After performing a course search, the search box in the upper right is misaligned in Firefox 3.0.8. It is correct in Internet Explorer 8.

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Jose Cedeno added a comment - 04/May/09 02:00 AM
I was trying to work on a patch for this issue, but I need a little more information. Is there a specific theme you are using? If it's another moodle site that is giving you a problem, do you have a link to the page showing the error?

Ryan Smith added a comment - 04/May/09 10:47 PM
Jose,

I am using the standard theme on a fresh Moodle install of version 1.9.4. To reproduce it, visit the 'All courses' link in the courses block, now you're at:

/moodle/course/index.php

See the search courses box? search for a course you know exists, then on the resulting search page you'll see the misaligned box in my screenshot I attached.


Jose Cedeno added a comment - 05/May/09 01:09 PM
This fixes the problem in firefox. I don't have Internet Explorer, so I cannot test the patch in that browser. I have tested in Firefox in the Mac as well as in Safari.

Jose Cedeno added a comment - 05/May/09 01:24 PM
Tim Hunt provided pointed out in my earlier patch that fieldsets are required for accessibility reasons and removing it, might break other themes. In this patch, I replaced the fieldset with a div.

Martín Langhoff added a comment - 01/Sep/09 01:11 AM
I have to agree with Tim – the fieldsets are required for accesibility. A div doesn't mean the same (in the 'semantic html' sense) as a fieldset tag.

If it gets "fixed" replacing a fieldset with a div, maybe FF attaches some CSS behaviour to the fieldset, which we need to override in the 'invisiblefieldset' css class?


Ryan Smith added a comment - 16/Feb/10 10:25 PM
I just tested this again in Firefox 3.6 and Moodle 1.9.7+ and it seems to be fixed. The search box is aligned properly. I think this can be closed.