(RTL) There is a delimitation line after the last app in listing pages

VERIFIED FIXED in 2015-07-21

Status

Marketplace
Consumer Pages
P4
trivial
VERIFIED FIXED
3 years ago
3 years ago

People

(Reporter: ValentinaP, Assigned: Trishul)

Tracking

Avenir
2015-07-21
x86_64
Windows 7
Points:
---

Details

Attachments

(1 attachment)

46 bytes, text/x-github-pull-request
Details | Review | Splinter Review
(Reporter)

Description

3 years ago
Steps to reproduce:
1. Load MP-stage page and set the language to Arabic https://marketplace.allizom.org/?lang=ar
2. Search for an app or go to apps listing page

Expected results:
There are no display or layout issues.

Actual results:
There is a vertical delimitation line after the last app.

Notes/Issues:
Verified on FF42(Win 7). Issue is reproduced on MP-stage.
Issue cannot be seen in LTR lang.
Screenshot for one app: http://screencast.com/t/WUCWnYx6jyu
Screencast for more apps: http://screencast.com/t/GoQZ0TXcUF76
Severity: normal → trivial
Priority: -- → P4
Target Milestone: 2015-07-14 → ---
(Assignee)

Comment 1

3 years ago
Created attachment 8633948 [details] [review]
Pull Request

Kindly review the PR,
Removed the border-left in last child (RTL)
https://github.com/tsl143/fireplace/commit/5e3616f48743fb943c9a100669e512524bab4de5
Assignee: nobody → trishul.goel
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2015-07-21
(Reporter)

Comment 3

3 years ago
Verified as fixed on MP-dev FF42(Win 7).
Postfix screenshot: http://screencast.com/t/JDBAeEtG
Closing bug.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.