Bug 572163 - [PENDING BACKPORT] ScrollBarVisibility.Hidden doesn't work
Summary: [PENDING BACKPORT] ScrollBarVisibility.Hidden doesn't work
Status: VERIFIED FIXED
Alias: None
Product: Moonlight
Classification: Mono
Component: plugin (show other bugs)
Version: 2.0.x
Hardware: x86 openSUSE 11.1
: P5 - None : Normal
Target Milestone: ---
Assignee: Swamp Workflow Management
QA Contact: Swamp Workflow Management
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 546068
  Show dependency tree / graph
 
Reported: 2010-01-20 08:19 UTC by calen chen
Modified: 2010-04-07 07:47 UTC (History)
0 users

See Also:
Found By: ---
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---

Attachments
ScrollViewerSample (20.00 KB, application/x-tar)
2010-01-20 08:19 UTC, calen chen 		Details
Patch which resolves the issue. (1.47 KB, patch)
2010-01-20 12:09 UTC, Alan McGovern 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description calen chen 2010-01-20 08:19:33 UTC 
Created attachment 337587 [details]
ScrollViewerSample

REPRO:

1. I have created a sample with Button and ScrollViewer controls
2. the sample can be build by mxap successful to create html and xap
3. run the html with Forefox on linux
4. click Button "Hidden Vertical"
5. click Button "Hidden Horizontal"

RESULTS:

after doing step4 and step5, Vertical scrollbar and horizontal scrollbar still are showing

EXPECTED RESULTS:

they shold be hidden

COMMENTS:

I got the moonlight2.0 xpi from moon-a11y team's hudson, here is the link: http://151.155.248.179:8010/job/_moonatkbridge_tag_xpi/
Comment 1 Alan McGovern 2010-01-20 12:09:47 UTC 
Created attachment 337638 [details]
Patch which resolves the issue.

We were missing a call to update the value of 'ComputedHorizontalScrollbarVisibility' and 'ComputedVerticalScrollbarVisibility' which are the values which the Scrollbars are databound to. The attached patch fixes your testcase.
Comment 2 Alan McGovern 2010-01-20 12:10:27 UTC 
This patch was committed in r149894 in moon trunk. Marking as pending backport.
Comment 3 Jeff Stedfast 2010-02-01 22:05:16 UTC 
just backported this patch
Comment 4 calen chen 2010-04-07 07:47:42 UTC 
Closed, big thanks!