Page display is not aligned properly in RTL page

[Efrat Nakash]

Chrome Version 41.0.2272.101 m (64-bit) on MSwindows 7

Page display is not aligned properly in RTL page, e.g. http://www.efratnakash.com/diaries_h.asp 

When you click on a link (on the book icon or on the hyperlinked text  ליומן המלא) - the display of the text in the same div changes, and becomes properly aligned. Only in the second click it opens the link. 

Same code (same CSS definition, with only different text) for LTR page displays properly, e.g. http://www.efratnakash.com/diaries_e.asp

The RTL page displays properly in other browsers.

Please help,

Efrat

[PhistucK]

This group discusses the Developer Tools feature of Google Chrome and not general web development issues.

You can search crbug.com for an existing issue and star it. If you cannot find one, file a new issue using the "New issue" link on the same page.

Please, do not add a "+1" or "Me too" or "Confirmed" (or similar) comment. It just wastes the time of Chrome engineers and sends unnecessary e-mails to all of the people who starred the issue.

You can reply with a link to the found or created issue and might get triaged (and fixed) faster.

Thank you.

A side note - wow, this page (the HTML itself has an inline script) is using some seriously old code and a bad form of browser/feature/whatever detection -

var clickmessage="כל הזכויות שמורות. אין להעתיק תמונות ללא הרשאה בכתב"

function disableclick(e) {

if (document.all) { // Should be replaced with if (window.attachEvent)

if (event.button==2||event.button==3) {

if (event.srcElement.tagName=="IMG"){

alert(clickmessage);

return false;

}

}

}

else if (document.layers) { // Obsolete, affects no one nowadays, this whole condition should be removed.

if (e.which == 3) {

alert(clickmessage);

return false;

}

}

else if (document.getElementById) // This is true everywhere, so the if (...) part can simply be omitted.

if (e.which==3&&e.target.tagName=="IMG")

setTimeout("alert(clickmessage)",0)

}

function associateimages(){

for(i=0;i<document.images.length;i++)

document.images[i].onmousedown=disableclick;

}

if (document.all){ // Should at least be replaced with if (window.attachEvent), but I am not sure what this is trying to workaround, so perhaps it is not needed at all.

document.onmousedown=disableclick

for (var i_tem = 0; i_tem < document.images.length; i_tem++)

document.images[i_tem].galleryimg='no'

}

else if (document.getElementById) // This is true everywhere, so the if (...) part can simply be omitted.

document.onmouseup=disableclick

else if (document.layers) // Obsolete, affects no one nowadays, this whole condition should be removed.

associateimages()

☆PhistucK

--
You received this message because you are subscribed to the Google Groups "Google Chrome Developer Tools" group.
To unsubscribe from this group and stop receiving emails from it, send an email to google-chrome-develo...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/google-chrome-developer-tools/f8728135-355f-4641-b5c4-62c67b20f35e%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

[Efrat Nakash]

Thanks PhistucK!

Will follow all your advises...