Issue Details (XML | Word | Printable)

Key: MDL-9977
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Critical Critical
Assignee: Urs Hunkler
Reporter: Daniele Cordella
Votes: 0
Watchers: 0
Operations

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

rounded corner theme problem with right to left languages

Created: 29/May/07 08:35 PM   Updated: 30/Jul/07 04:52 AM
Component/s: Themes
Affects Version/s: 1.9
Fix Version/s: 1.9

File Attachments: None
Image Attachments:

1. Immagine 1.png
(13 kB)
Issue Links:
Blockers
 

Participants: Daniele Cordella, Mauno Korpelainen and Urs Hunkler
Security Level: None
Resolved date: 30/Jul/07
Affected Branches: MOODLE_19_STABLE
Fixed Branches: MOODLE_19_STABLE


 Description  « Hide
go to http://test.moodle.com/ and select "fa" in the language pop up menu (the first after Euskara (eu))

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Urs Hunkler added a comment - 11/Jun/07 03:28 PM - edited
I am working on the rtl issue. Hope to find a flexible solution. You may look at the linked issue.

Mauno Korpelainen added a comment - 23/Jul/07 12:37 AM
Hi Urs,

I tested this with minor changes to lib/custom_corners_lib.php, lib/weblib.php and theme/custom_corners/user_styles.css

I changed

<div class="wrap">
<div class="bt"><div></div></div>
....
<div class="bb"><div></div></div>
</div>

to a "three image box" - left image, border image and right image for top and bottom of the box using

<DIV id="box">
<DIV id="box_t">
<DIV id="box_tl">
<DIV id="box_tr"></DIV></DIV></DIV>
....
<DIV id=box_b>
<DIV id=box_bl>
<DIV id=box_br></DIV></DIV></DIV>
</DIV>

It worked fine with both rtl and ltr languages and without a need to use styles_rtl.css at all or any other fixes. I will test some more (first I will try the order tl-t-tr...bl-b-br with some new css because I did not test that yet) and send you some mail later...


Urs Hunkler added a comment - 30/Jul/07 04:52 AM
Solved with the fix for MDL-10075.