Mahara logo is cut off at the bottom in IE 11

Bug #1726655 reported by Robert Lyon
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
High
Robert Lyon
17.04
Fix Released
High
Unassigned
17.10
Fix Released
High
Robert Lyon
18.04
Fix Released
High
Robert Lyon

Bug Description

In IE the base of the Mahara logo is being hidden

I believe it is due to the fact that the image has a height of 45px but the container it is in is floated and so lacks height.

I tried adding 60px height to the containing <a> tag and it looked ok

Will make a patch so people can test

Robert Lyon (robertl-9)
Changed in mahara:
status: New → Confirmed
importance: Undecided → Medium
assignee: nobody → Robert Lyon (robertl-9)
Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "master" branch: https://reviews.mahara.org/8177

no longer affects: mahara/18.04
Revision history for this message
Mahara Bot (dev-mahara) wrote :

Patch for "17.04_STABLE" branch: https://reviews.mahara.org/8253

Revision history for this message
Niranjan (niranjan528) wrote :

URL: https://bugs.launchpad.net/mahara/+bug/1726655

Bug description: Mahara logo is cut off at the bottom in IE11.

Environment to be tested: Master
Browser to be tested: IE 11

Precondition:
The institution 'Institution A' exists and the user is a admin in that institution.

Steps to reproduce:

1. Open IE 11 and enter the URL to the Mahara instance.

Expected result: Mahara logo should be displayed properly without any
cut off.

2. Log in as ‘Institution admin’.

Expected result: Mahara logo should be displayed properly without any cut off.

3. Navigate to Administration menu > Institutions > Settings.
4. Click the ‘Settings’ button for ‘Institution A’.
5. Upload a logo and Click the ‘Submit’ button.
6. Log out and Log in back as ‘Institution admin’.

Expected result: Mahara logo should be displayed properly without any cut off.

7. Compare the Mahara logo with another browser and verify if the dimensions are aligned properly.

Expected result: Mahara logo should have the same dimensions as the other browser.

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/8177
Committed: https://git.mahara.org/mahara/mahara/commit/a65e3cd952dea467d1c2d899b88fb35d5601a96a
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit a65e3cd952dea467d1c2d899b88fb35d5601a96a
Author: Robert Lyon <email address hidden>
Date: Tue Oct 24 14:37:59 2017 +1300

Bug 1726655: Allow more space for logo in IE11

behatnotneeded

Change-Id: If68427764654a1f92df89453515ee2c3f710bb61
Signed-off-by: Robert Lyon <email address hidden>

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "17.10_STABLE" branch: https://reviews.mahara.org/8273

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/8253
Committed: https://git.mahara.org/mahara/mahara/commit/79c2d745121a7fabf93acbf17399de6a2838ec39
Submitter: Robert Lyon (<email address hidden>)
Branch: 17.04_STABLE

commit 79c2d745121a7fabf93acbf17399de6a2838ec39
Author: Pat Kira <email address hidden>
Date: Thu Nov 2 12:45:09 2017 +1100

Bug 1726655: Allow more space for logo in IE11

behatnotneeded

Change-Id: I9be88740d6e93297a7895156012b37314d706b75

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Reviewed: https://reviews.mahara.org/8273
Committed: https://git.mahara.org/mahara/mahara/commit/236a514ccd6f6550851e07be7f2a8c085ff3184b
Submitter: Robert Lyon (<email address hidden>)
Branch: 17.10_STABLE

commit 236a514ccd6f6550851e07be7f2a8c085ff3184b
Author: Robert Lyon <email address hidden>
Date: Tue Oct 24 14:37:59 2017 +1300

Bug 1726655: Allow more space for logo in IE11

behatnotneeded

Change-Id: If68427764654a1f92df89453515ee2c3f710bb61
Signed-off-by: Robert Lyon <email address hidden>
(cherry picked from commit a65e3cd952dea467d1c2d899b88fb35d5601a96a)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.