Open Bug 483798 Opened 15 years ago Updated 2 years ago

[RTL] no horizontal scrollbar in RTL mode

Categories

(Core :: Layout, defect)

Product:
Core
Component:
Layout
Type:
defect

Tracking

()

People

(Reporter: qetzal, Unassigned)

References

Details

(Keywords: regression, rtl, testcase, Whiteboard: [post-2.0])

Attachments

(2 files)

'test.html' file
479 bytes, text/html
Details
Patch (v1)
1.06 KB, patch
roc
: review-
Details | Diff | Splinter Review 
User-Agent:       Opera/9.63 (X11; FreeBSD 6.3-RELEASE i386; U; en) Presto/2.1.1
Build Identifier: Mozilla/5.0 (X11; U; FreeBSD i386; en-US; rv:1.9.0.5) Gecko/2009020212 Firefox/3.0.5

When HTML page in is right-to-left mode and some CSS rules are applied, FireFox doesn't show the horizontal scrollbar when it is necessary.

Reproducible: Always

Steps to Reproduce:
1. Open the attached 'test.html' file. 
2. Change the size of your FireFox window, decrease it significally. 
3. You will see that there is now horizontal scrollbar.
Actual Results:  
No horizontal scrollbar.

Expected Results:  
I expect horizontal scrollbar.
Attached file 'test.html' file — 

    
    

    
  
I mean: "3. You will see that there is _no_ horizontal scrollbar."
Sorry for the typo.

    
    

    
  
Confirmed on Windows XP/Vista. It is a regression from Firefox 2.
Status: UNCONFIRMED → NEW
Component: General → Layout
Ever confirmed: true
Keywords: regression, 
          
            testcase
OS: FreeBSD → All
Product: Firefox → Core
QA Contact: general → layout
Hardware: x86 → All
Version: unspecified → Trunk
Keywords: rtl

    
    

    
  
Confirmed on Windows 7 FireFox 3.6.8, and there are many RTL Sites facing this issue. i.e.: http://www.shabayek.com/blog/

    
    

    
  

      
      
      
      

        Attached patch
          
          
        Patch (v1)
        — 
          — Splinter Review
      

    


  
I couldn't believe my eyes when I saw this code!  As far as I tested, this patch works and doesn't break anything else.
Assignee: nobody → ehsan
Status: NEW → ASSIGNED

        Attachment #471329 -
        Flags: review?(roc)

    
    

    
  
I don't think this patch is correct.

Consider a testcase like
<body style="direction: rtl">
  <div style="position:absolute; left:-100%; width:300%; height:100px; background:blue;"></div>

With your patch, I think we'll try to have the scrollbar long enough to scroll in both directions, with the default position in the middle. Or something like that. We don't want that. In LTR documents, you can't scroll to the left of 0. In RTL documents you shouldn't be able to scroll to the right of the viewport width.

I think this is actually a layout bug. In a narrow window, this wide element should be placed so that its x-coordinate is negative.

Is this a regression? If so, a regression window would be helpful.

    
    

    
  
Comment on attachment 471329 [details] [diff] [review]
Patch (v1)

See comment

        Attachment #471329 -
        Flags: review?(roc) → review-

    
  
Whiteboard: [post-2.0]

    
    

    
  
in below address when i click on "Switch Text Direction" in right-click, still not show horizontal scroll bar.
http://forums.sakhtafzarmag.com/

    
  
Assignee: ehsan → nobody
Status: ASSIGNED → NEW

    
  
Severity: normal → S3

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: