Loading...

XML

Word

Printable

Details

Type: Bug
Resolution: Fixed
Priority: Major
Fix Version/s: 2.5.5, 2.6.2
Affects Version/s: 2.5, 2.5.4, 2.6.1
Component/s: Messages, Themes
Labels:
- ci
- rtl
- triaged

Affected Branches:
MOODLE_25_STABLE, MOODLE_26_STABLE
Fixed Branches:
MOODLE_25_STABLE, MOODLE_26_STABLE
Epic Link:
Clean as default
Pull from Repository:
git://github.com/FMCorz/moodle.git
Pull Main Branch:
~~MDL-39683~~-master
Pull Main Diff URL:
https://github.com/FMCorz/moodle/compare/12efa52...MDL-39683-master
Testing Instructions:
Hide

Test in Clean and Standard

Go into your user's Messaging page.

Send a few messages to any user.

Login as that other user and send some messages to yourself (you can reply your previous message).

Go back being you.

Switch language to Hebrew (or any other RTL language).

Make sure that your personal picture is displayed on the right side of the screen and the other user's picture on the left.

Make sure that the messages are aligned the name of the user who sent them.
Show
Test in Clean and Standard Go into your user's Messaging page. Send a few messages to any user. Login as that other user and send some messages to yourself (you can reply your previous message). Go back being you. Switch language to Hebrew (or any other RTL language). Make sure that your personal picture is displayed on the right side of the screen and the other user's picture on the left. Make sure that the messages are aligned the name of the user who sent them.

Description

There are some alignment issues on /message/index.php. Not sure if this is due to using an RTL language, the Clean theme or the combination of both. Attaching screenshots of English/Standard Vs Arabic/Clean.

Attachments

- Sort By Name
- Sort By Date
- Ascending
- Descending
- Thumbnails
- List
- Download All

after.png
244 kB
20/Aug/13 2:12 PM
before.png
244 kB
20/Aug/13 2:12 PM
LTRStandardMessaging.png
170 kB
15/May/13 11:25 AM
message.png
34 kB
15/Jan/14 10:56 AM
RTLCleanMessaging.png
157 kB
15/May/13 11:25 AM

Activity

People

Assignee:: Frédéric Massart

Reporter:: Andrew Davis

Peer reviewer:: Rossiani Wijaya

Integrator:: Damyon Wiese

Tester:: Sam Hemelryk

Participants:: Andrew Davis, CiBoT, Damyon Wiese, Dan Poltawski, Eloy Lafuente (stronk7), Frédéric Massart, Rossiani Wijaya, Sam Hemelryk

Votes:: 0 Vote for this issue

Watchers:: 5 Start watching this issue

Dates

Created:: 15/May/13 11:23 AM

Updated:: 23/Jan/14 8:47 PM

Resolved:: 23/Jan/14 8:47 PM

Fix Release Date:: 10/Mar/14