Add an adaptive visual density static function, and add it to the sample app. #51921
Conversation
fluttergithubbot
added
f: material design
|
/cc @csells |
|
Gold has detected one or more untriaged digests on patchset 2. |
gspencergoog
force-pushed
the
adaptive_density
branch
from
b477b76 to
347877a
Compare
zanderso
removed
the
tool
| // This makes the visual density adapt to the platform that you run | ||
| // the app on. For desktop platforms, the controls will be smaller and | ||
| // closer together (more dense) than on mobile platforms. | ||
| visualDensity: VisualDensity.adaptivePlatformDensity, |
There was a problem hiding this comment.
why do we need this? isn't being adaptive the default?
There was a problem hiding this comment.
No, being adaptive isn't the default, because the Material Design team wanted the "mobile" density to be the default. Their reasoning was that some desktop systems have touch screens, and so those screens should have 48x48 touch targets, so as not to compromise accessibility.
We had anecdotal evidence, however, that developers would like the option to have it be adaptive, so this was the compromise that we came up with.
7 participants
Description
Adds a
VisualDensity.adaptivePlatformDensitystatic function that returns different values for visual density based on thedefaultTargetPlatform. Returnscompactfor desktop platforms, and a default visual density for other platforms.Issues
Tests
Breaking Change