Bug 145021

Summary: [CSS Grid Layout] Add scrollbar width in intrinsic logical widths computation
Product: WebKit Reporter: Manuel Rego Casasnovas <rego>
Component: Layout and RenderingAssignee: Manuel Rego Casasnovas <rego>
Status: RESOLVED FIXED    
Severity: Normal CC: cdumez, commit-queue, darin, esprehn+autocc, glenn, hyatt, jfernandez, kondapallykalyan, svillar
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 60731    
Attachments:
Description Flags
Patch none

Manuel Rego Casasnovas
Reported 2015-05-14 15:03:07 PDT
Currently there's the following FIXME in RenderGrid::computeIntrinsicLogicalWidths(): // FIXME: This should add in the scrollbarWidth (e.g. see RenderFlexibleBox). We should add the scrollbar width when we calculate the intrinsic with of the grid container.
Attachments
Patch (5.13 KB, patch)
2015-05-14 15:04 PDT, Manuel Rego Casasnovas 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Manuel Rego Casasnovas
Comment 1 2015-05-14 15:04:56 PDT
Created attachment 253148 [details] Patch
WebKit Commit Bot
Comment 2 2015-05-17 23:17:05 PDT
Comment on attachment 253148 [details] Patch Clearing flags on attachment: 253148 Committed r184446: <http://trac.webkit.org/changeset/184446>
WebKit Commit Bot
Comment 3 2015-05-17 23:17:12 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.