Remove superfluous <td> in résumé section

Bug #1844444 reported by Kristina Hoeppner
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Medium
Unassigned

Bug Description

When you add employment history, the number to indicate how many attachments are there for one job is wrapped into 2 <td> when there should only be one like in "Education history".

To replicate:

1. Add 1 item in "Education history" in your résumé and attach a file.
2. Add 1 item in "Employment history" in your résumé and attach a file.
3. Inspect the HTML code for the attachment number with the dev tool console.
Expected result: The number should only be in 1 <td> like so: <td style="text-align:center">1</td>
Actual result: It is wrapped in 2 <td>: <td><td style="text-align:center">1</td></td>

Tags: bite-sized
Revision history for this message
Kristina Hoeppner (kris-hoeppner) wrote :
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/10952

Changed in mahara:
assignee: Doris Tam (doristam) → nobody
milestone: none → 20.10.0
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

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

commit 7916358b5c8a9b46f56745bff5a7d09aca7653ce
Author: Evonne <email address hidden>
Date: Mon May 4 08:17:33 2020 +1200

Bug 1844444 Remove superfluous <td> in r��sum�� section

Changed second <td> to <span> instead to avoid the extra border
on top.

Change-Id: Ia9a81629ea7cbc26b2ea95cb623dff5d0173a7dd

Robert Lyon (robertl-9)
Changed in mahara:
status: Confirmed → Fix Committed
Robert Lyon (robertl-9)
Changed in mahara:
status: Fix Committed → Fix Released
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.