Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[M3] TextField "hovered" has priority over "focused". Filled "focused" and all "focused" error border is 1 dp, should be 2dp. #113329

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
rydmike opened this issue Oct 12, 2022 · 5 comments · Fixed by #113776
Assignees
Labels
a: text input Entering text in a text field or keyboard related problems f: material design flutter/packages/flutter/material repository. found in release: 3.3 Found to occur in 3.3 found in release: 3.4 Found to occur in 3.4 framework flutter/packages/flutter repository. See also f: labels. has reproducible steps The issue has been confirmed reproducible and is ready to work on r: fixed Issue is closed as already fixed in a newer version

Comments

@rydmike
Copy link
Contributor

rydmike commented Oct 12, 2022

Background

This issue is filed based on my comment here #110878 (comment) and reply here #110878 (comment) by @HansMuller to post my audit findings of the M3 TextField decoration.

Issue Description

In master channel (v. 3.5.0-6.0.pre.19) when enabling Material 3 and using a state of "focused" + "hovered". A state you often end up with on desktop when you move the cursor to a text field and focus it, since it will effectively be both focused and hovered then if cursor remains on it. In such a scenario the M3 TextField will get partially "focused", but mostly "hovered" visual design.

The M3 guide does not explicitly say what the result should be since the combination is not included in spec, but it looks like an oversight and not right. Especially considering that a TextField that is both "focused" and "hovered", now suddenly looks less prominent than a TextField that is only "focused", this is probably not intended.

It is here assumed implicitly from state presentation order in the M3 guide, that if both "Focused" and "Hovered" states are present, that "Focused" should have priority. This would look better and be consistent with past M2 behavior. Currently the M3 TextField behaves a bit unexpectedly since it changes its look if you have mouse clicked on it with desktop, to focus it, it is then also "hovered" since your cursor is on it. In current M3 InputDecorator it then gets the "hovered" state, since it has precedence over "focused" in current code. Then if you move the cursor away from the Textfield it remains "Focused", as it should, but is no longer "Hovered", the style of the field will then change. It probably should not. Often you just move the mouse course away from the field since it might be in the way of the input caret and text, you do not expect the style of the input field to change when you do so.

When using filled input border, the focused input border is 1dp, same for focused error border with both M3 border types. Based on web published M3 guide and spec, focused filled input border and focused error borders should be 2dp (https://m3.material.io/components/text-fields/specs).

Screen Shot 2022-10-12 at 15 34 52
Screen Shot 2022-10-12 at 15 36 00

Actual and Expected Results

Filled Underline Border

State & Comment Actual Expected (M3 spec)
OK Normal Screen Shot 2022-10-12 at 15 43 35 Screen Shot 2022-10-12 at 15 46 35
FAIL Focus, underline is 1dp, should be 2dp Screen Shot 2022-10-12 at 15 49 29 Screen Shot 2022-10-12 at 15 50 11
OK Hover Screen Shot 2022-10-12 at 15 52 45 Screen Shot 2022-10-12 at 15 53 28
FAIL Focused+Hover, bg gets focus, rest hover state. Prefer Focused!? Screen Shot 2022-10-12 at 16 33 01 Screen Shot 2022-10-12 at 15 50 11
OK Error Screen Shot 2022-10-12 at 16 15 02 Screen Shot 2022-10-12 at 16 05 27
FAIL Error+Focus 1dp instead of 2dp underline Screen Shot 2022-10-12 at 16 15 33 Screen Shot 2022-10-12 at 16 08 59
OK Error+Hover Screen Shot 2022-10-12 at 16 15 58 Screen Shot 2022-10-12 at 16 10 53
FAIL Error+Focus+Hover, bg gets focus, rest hover. Prefer focused!? Screen Shot 2022-10-12 at 16 16 25 Screen Shot 2022-10-12 at 16 08 59

Unfilled Outlined Border

State & Comment Actual Expected (M3 spec)
OK Normal Screen Shot 2022-10-12 at 17 07 11 Screen Shot 2022-10-12 at 17 01 11
OK Focused, is 2dp Screen Shot 2022-10-12 at 17 07 50 Screen Shot 2022-10-12 at 17 01 28
OK Hover Screen Shot 2022-10-12 at 17 11 22 Screen Shot 2022-10-12 at 17 01 44
FAIL Focus+Hover, gets all hover state. Prefer focused!? Screen Shot 2022-10-12 at 17 12 08 Screen Shot 2022-10-12 at 17 01 28
OK Error Screen Shot 2022-10-12 at 17 13 35 Screen Shot 2022-10-12 at 17 04 47
FAIL Error+Focus 1dp instead of 2dp Screen Shot 2022-10-12 at 17 15 52 Screen Shot 2022-10-12 at 17 04 59
OK Error+Hover Screen Shot 2022-10-12 at 17 17 51 Screen Shot 2022-10-12 at 17 05 12
FAIL Error+Focus+Hover, gets hover state. Prefer focused!? Screen Shot 2022-10-12 at 17 18 15 Screen Shot 2022-10-12 at 17 04 59

Summary of Issues and Solution Proposal

BorderSide 2dp

The BorderSide should be 2dp and not 1dp for focused border side, also in filled TextField and for focused error state with both border types, at least based on M3 web specification. This is missing in input_decorator.dart:
Screen Shot 2022-10-12 at 16 35 32
Above the BorderSide.width: 2.0 for "focused" state is also missing inside "error" state for both activeIndicatorBorder and outlineBorder

Order of MaterialState Evaluation

In M3 InputDecorator, whenever states.contains "focused" and "hovered" is used, contains "focused" should be evaluated before "hovered". This will give "focused" state precedence over "hovered", like it should be and resolve the current odd "focused+hovered" state. Currently we have this in input_decorator.dart:

Screen Shot 2022-10-12 at 16 41 37
The order matters. The above is only one example, the order is the same wherever focused and hovered states are evaluated.

Demo Application

A simple demo application using the M3 spec baseline ColorScheme that can show underline TextField and outline TextField and errors states is supplied. It can also demonstrate the issues in dark mode, they were not shown above, but they are equivalent:

Light mode demo Dark mode demo
Screen Shot 2022-10-12 at 16 58 33 Screen Shot 2022-10-12 at 16 58 59

Reproduction Sample

The demo code for the sample app can be found in GIST here, and is also supplied below.

To actually see the wrong results, manual testing on a desktop or web build is required.

Code sample
// MIT License
//
// Copyright (c) 2022 Mike Rydstrom
//
// Permission is hereby granted, free of charge, to any person obtaining a copy
// of this software and associated documentation files (the "Software"), to deal
// in the Software without restriction, including without limitation the rights
// to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
// copies of the Software, and to permit persons to whom the Software is
// furnished to do so, subject to the following conditions:
//
// The above copyright notice and this permission notice shall be included in all
// copies or substantial portions of the Software.
//
// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
// IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
// FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
// AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
// LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
// OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
// SOFTWARE.

import 'package:flutter/material.dart';

// App breakpoints
const double phoneWidthBreakpoint = 550;
const double phoneHeightBreakpoint = 600;

// Used as M3 seed color, the M3 baseline generating seed color.
// This color is also used for the TextField spec examples.
const Color seedColor = Color(0xFF6750A4);

// Make a seed generated M3 light mode ColorScheme.
final ColorScheme myLightScheme = ColorScheme.fromSeed(
  brightness: Brightness.light,
  seedColor: seedColor,
);

// Make a seed generated M3 light mode ColorScheme.
final ColorScheme myDarkScheme = ColorScheme.fromSeed(
  brightness: Brightness.dark,
  seedColor: seedColor,
);

// A simple custom theme
ThemeData myTheme(Brightness mode) => ThemeData.from(
      colorScheme: mode == Brightness.light ? myLightScheme : myDarkScheme,
      useMaterial3: true,
    ).copyWith(
      // Use standard so desktop sizes matches mobile for this example.
      visualDensity: VisualDensity.standard,
    );

void main() {
  runApp(const TextFieldDemoApp());
}

class TextFieldDemoApp extends StatefulWidget {
  const TextFieldDemoApp({super.key});

  @override
  State<TextFieldDemoApp> createState() => _TextFieldDemoAppState();
}

class _TextFieldDemoAppState extends State<TextFieldDemoApp> {
  bool useMaterial3 = false;
  ThemeMode themeMode = ThemeMode.light;

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      debugShowCheckedModeBanner: false,
      themeMode: themeMode,
      theme: myTheme(Brightness.light),
      darkTheme: myTheme(Brightness.dark),
      home: Scaffold(
        appBar: AppBar(
          title: const Text(('TextField States Issue?')),
          actions: [
            IconButton(
              icon: themeMode == ThemeMode.dark
                  ? const Icon(Icons.wb_sunny_outlined)
                  : const Icon(Icons.wb_sunny),
              onPressed: () {
                setState(() {
                  if (themeMode == ThemeMode.light) {
                    themeMode = ThemeMode.dark;
                  } else {
                    themeMode = ThemeMode.light;
                  }
                });
              },
              tooltip: "Toggle brightness",
            ),
          ],
        ),
        body: const HomePage(),
      ),
    );
  }
}

class HomePage extends StatelessWidget {
  const HomePage({
    super.key,
  });

  @override
  Widget build(BuildContext context) {
    return ListView(
      padding: const EdgeInsets.symmetric(horizontal: 16),
      children: [
        const SizedBox(height: 8),
        Text(
          'Material 3 TextField Manual Test',
          style: Theme.of(context).textTheme.headlineSmall,
        ),
        const SizedBox(height: 8),
        const TextInputStatesDemo(),
        const SizedBox(height: 8),
        const ShowColorSchemeColors(),
      ],
    );
  }
}

class TextInputStatesDemo extends StatefulWidget {
  const TextInputStatesDemo({Key? key}) : super(key: key);

  @override
  State<TextInputStatesDemo> createState() => _TextInputStatesDemoState();
}

class _TextInputStatesDemoState extends State<TextInputStatesDemo> {
  late TextEditingController _textController1;
  late TextEditingController _textController2;
  bool _errorState1 = false;
  bool _errorState2 = false;

  @override
  void initState() {
    super.initState();
    _textController1 = TextEditingController();
    _textController2 = TextEditingController();
  }

  @override
  void dispose() {
    _textController1.dispose();
    _textController2.dispose();
    super.dispose();
  }

  @override
  Widget build(BuildContext context) {
    return Column(
      children: <Widget>[
        TextField(
          onChanged: (String text) {
            setState(() {
              if (text.contains('a') | text.isEmpty) {
                _errorState1 = false;
              } else {
                _errorState1 = true;
              }
            });
          },
          key: const Key('TextField1'),
          controller: _textController1,
          decoration: InputDecoration(
            prefixIcon: const Icon(Icons.search),
            suffixIcon: const Icon(Icons.cancel_outlined),
            // border: const UnderlineInputBorder(),
            filled: true,
            hintText: 'Write something...',
            labelText: 'Text entry',
            errorText: _errorState1
                ? "Any entry without an 'a' will trigger this error"
                : null,
          ),
        ),
        const SizedBox(height: 16),
        TextField(
          onChanged: (String text) {
            setState(() {
              if (text.contains('a') | text.isEmpty) {
                _errorState2 = false;
              } else {
                _errorState2 = true;
              }
            });
          },
          key: const Key('TextField2'),
          controller: _textController2,
          decoration: InputDecoration(
            prefixIcon: const Icon(Icons.search),
            suffixIcon: const Icon(Icons.cancel_outlined),
            border: const OutlineInputBorder(),
            hintText: 'Write something...',
            labelText: 'Another text entry',
            errorText: _errorState2
                ? "Any entry without an 'a' will trigger this error"
                : null,
          ),
        ),
      ],
    );
  }
}

/// Draw a number of boxes showing the colors of key theme color properties
/// in the ColorScheme of the inherited ThemeData and its color properties.
class ShowColorSchemeColors extends StatelessWidget {
  const ShowColorSchemeColors({super.key, this.onBackgroundColor});

  /// The color of the background the color widget are being drawn on.
  ///
  /// Some of the theme colors may have semi transparent fill color. To compute
  /// a legible text color for the sum when it shown on a background color, we
  /// need to alpha merge it with background and we need the exact background
  /// color it is drawn on for that. If not passed in from parent, it is
  /// assumed to be drawn on card color, which usually is close enough.
  final Color? onBackgroundColor;

  // Return true if the color is light, meaning it needs dark text for contrast.
  static bool _isLight(final Color color) =>
      ThemeData.estimateBrightnessForColor(color) == Brightness.light;

  // Return true if the color is dark, meaning it needs light text for contrast.
  static bool _isDark(final Color color) =>
      ThemeData.estimateBrightnessForColor(color) == Brightness.dark;

  // On color used when a theme color property does not have a theme onColor.
  static Color _onColor(final Color color, final Color bg) =>
      _isLight(Color.alphaBlend(color, bg)) ? Colors.black : Colors.white;

  @override
  Widget build(BuildContext context) {
    final ThemeData theme = Theme.of(context);
    final ColorScheme colorScheme = theme.colorScheme;
    final bool isDark = colorScheme.brightness == Brightness.dark;
    final bool useMaterial3 = theme.useMaterial3;

    final MediaQueryData media = MediaQuery.of(context);
    final bool isPhone = media.size.width < phoneWidthBreakpoint ||
        media.size.height < phoneHeightBreakpoint;
    final double spacing = isPhone ? 3 : 6;

    // Grab the card border from the theme card shape
    ShapeBorder? border = theme.cardTheme.shape;
    // If we had one, copy in a border side to it.
    if (border is RoundedRectangleBorder) {
      border = border.copyWith(
        side: BorderSide(
          color: theme.dividerColor,
          width: 1,
        ),
      );
      // If
    } else {
      // If border was null, make one matching Card default, but with border
      // side, if it was not null, we leave it as it was.
      border ??= RoundedRectangleBorder(
        borderRadius: BorderRadius.all(Radius.circular(useMaterial3 ? 12 : 4)),
        side: BorderSide(
          color: theme.dividerColor,
          width: 1,
        ),
      );
    }

    // Get effective background color.
    final Color background =
        onBackgroundColor ?? theme.cardTheme.color ?? theme.cardColor;

    // Warning label for scaffold background when it uses to much blend.
    final String surfaceTooHigh = isDark
        ? _isLight(theme.colorScheme.surface)
            ? '\nTOO HIGH'
            : ''
        : _isDark(theme.colorScheme.surface)
            ? '\nTOO HIGH'
            : '';

    // Warning label for scaffold background when it uses to much blend.
    final String backTooHigh = isDark
        ? _isLight(theme.colorScheme.background)
            ? '\nTOO HIGH'
            : ''
        : _isDark(theme.colorScheme.background)
            ? '\nTOO HIGH'
            : '';

    // Wrap this widget branch in a custom theme where card has a border outline
    // if it did not have one, but retains in ambient themed border radius.
    return Theme(
      data: Theme.of(context).copyWith(
        cardTheme: CardTheme.of(context).copyWith(
          elevation: 0,
          shape: border,
        ),
      ),
      child: Column(
        crossAxisAlignment: CrossAxisAlignment.start,
        children: <Widget>[
          Padding(
            padding: const EdgeInsets.symmetric(vertical: 8),
            child: Text(
              'ColorScheme Colors (M3 Baseline colors)',
              style: theme.textTheme.titleMedium,
            ),
          ),
          Wrap(
            alignment: WrapAlignment.start,
            crossAxisAlignment: WrapCrossAlignment.center,
            spacing: spacing,
            runSpacing: spacing,
            children: <Widget>[
              ColorCard(
                label: 'Primary',
                color: colorScheme.primary,
                textColor: colorScheme.onPrimary,
              ),
              ColorCard(
                label: 'on\nPrimary',
                color: colorScheme.onPrimary,
                textColor: colorScheme.primary,
              ),
              ColorCard(
                label: 'Primary\nContainer',
                color: colorScheme.primaryContainer,
                textColor: colorScheme.onPrimaryContainer,
              ),
              ColorCard(
                label: 'onPrimary\nContainer',
                color: colorScheme.onPrimaryContainer,
                textColor: colorScheme.primaryContainer,
              ),
              ColorCard(
                label: 'Secondary',
                color: colorScheme.secondary,
                textColor: colorScheme.onSecondary,
              ),
              ColorCard(
                label: 'on\nSecondary',
                color: colorScheme.onSecondary,
                textColor: colorScheme.secondary,
              ),
              ColorCard(
                label: 'Secondary\nContainer',
                color: colorScheme.secondaryContainer,
                textColor: colorScheme.onSecondaryContainer,
              ),
              ColorCard(
                label: 'on\nSecondary\nContainer',
                color: colorScheme.onSecondaryContainer,
                textColor: colorScheme.secondaryContainer,
              ),
              ColorCard(
                label: 'Tertiary',
                color: colorScheme.tertiary,
                textColor: colorScheme.onTertiary,
              ),
              ColorCard(
                label: 'on\nTertiary',
                color: colorScheme.onTertiary,
                textColor: colorScheme.tertiary,
              ),
              ColorCard(
                label: 'Tertiary\nContainer',
                color: colorScheme.tertiaryContainer,
                textColor: colorScheme.onTertiaryContainer,
              ),
              ColorCard(
                label: 'on\nTertiary\nContainer',
                color: colorScheme.onTertiaryContainer,
                textColor: colorScheme.tertiaryContainer,
              ),
              ColorCard(
                label: 'Error',
                color: colorScheme.error,
                textColor: colorScheme.onError,
              ),
              ColorCard(
                label: 'on\nError',
                color: colorScheme.onError,
                textColor: colorScheme.error,
              ),
              ColorCard(
                label: 'Error\nContainer',
                color: colorScheme.errorContainer,
                textColor: colorScheme.onErrorContainer,
              ),
              ColorCard(
                label: 'onError\nContainer',
                color: colorScheme.onErrorContainer,
                textColor: colorScheme.errorContainer,
              ),
              ColorCard(
                label: 'Background$backTooHigh',
                color: colorScheme.background,
                textColor: colorScheme.onBackground,
              ),
              ColorCard(
                label: 'on\nBackground',
                color: colorScheme.onBackground,
                textColor: colorScheme.background,
              ),
              ColorCard(
                label: 'Surface$surfaceTooHigh',
                color: colorScheme.surface,
                textColor: colorScheme.onSurface,
              ),
              ColorCard(
                label: 'on\nSurface',
                color: colorScheme.onSurface,
                textColor: colorScheme.surface,
              ),
              ColorCard(
                label: 'Surface\nVariant',
                color: colorScheme.surfaceVariant,
                textColor: colorScheme.onSurfaceVariant,
              ),
              ColorCard(
                label: 'onSurface\nVariant',
                color: colorScheme.onSurfaceVariant,
                textColor: colorScheme.surfaceVariant,
              ),
              ColorCard(
                label: 'Outline',
                color: colorScheme.outline,
                textColor: colorScheme.background,
              ),
              ColorCard(
                label: 'Outline\nVariant',
                color: colorScheme.outlineVariant,
                textColor: colorScheme.onBackground,
              ),
              ColorCard(
                label: 'Shadow',
                color: colorScheme.shadow,
                textColor: _onColor(colorScheme.shadow, background),
              ),
              ColorCard(
                label: 'Scrim',
                color: colorScheme.scrim,
                textColor: _onColor(colorScheme.scrim, background),
              ),
              ColorCard(
                label: 'Inverse\nSurface',
                color: colorScheme.inverseSurface,
                textColor: colorScheme.onInverseSurface,
              ),
              ColorCard(
                label: 'onInverse\nSurface',
                color: colorScheme.onInverseSurface,
                textColor: colorScheme.inverseSurface,
              ),
              ColorCard(
                label: 'Inverse\nPrimary',
                color: colorScheme.inversePrimary,
                textColor: colorScheme.inverseSurface,
              ),
              ColorCard(
                label: 'Surface\nTint',
                color: colorScheme.surfaceTint,
                textColor: colorScheme.onPrimary,
              ),
            ],
          ),
        ],
      ),
    );
  }
}

/// A [SizedBox] with a [Card] and string text in it. Used in this demo to
/// display theme color boxes.
///
/// Can specify label text color and background color.
class ColorCard extends StatelessWidget {
  const ColorCard({
    super.key,
    required this.label,
    required this.color,
    required this.textColor,
    this.size,
  });

  final String label;
  final Color color;
  final Color textColor;
  final Size? size;

  @override
  Widget build(BuildContext context) {
    const double fontSize = 11;
    const Size effectiveSize = Size(86, 58);

    return SizedBox(
      width: effectiveSize.width,
      height: effectiveSize.height,
      child: Card(
        margin: EdgeInsets.zero,
        clipBehavior: Clip.antiAlias,
        color: color,
        child: Center(
          child: Text(
            label,
            style: TextStyle(color: textColor, fontSize: fontSize),
            textAlign: TextAlign.center,
          ),
        ),
      ),
    );
  }
}

Used Flutter Version

Flutter doctor
flutter doctor -v
[✓] Flutter (Channel master, 3.5.0-6.0.pre.19, on macOS 12.5.1 21G83 darwin-arm64, locale en-US)
    • Flutter version 3.5.0-6.0.pre.19 on channel master at /Users/rydmike/fvm/versions/master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision aab696c236 (5 hours ago), 2022-10-12 02:59:35 -0400
    • Engine revision 481f201689
    • Dart version 2.19.0 (build 2.19.0-302.0.dev)
    • DevTools version 2.18.0

[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at /Users/rydmike/Library/Android/sdk
    • Platform android-33, build-tools 32.1.0-rc1
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13F17a
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[✓] IntelliJ IDEA Community Edition (version 2022.1.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 67.1.4
    • Dart plugin version 221.5591.58

[✓] VS Code (version 1.72.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 12.5.1 21G83 darwin-arm64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 106.0.5249.103

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

@exaby73 exaby73 added the in triage Presently being triaged by the triage team label Oct 13, 2022
@exaby73
Copy link
Member

exaby73 commented Oct 13, 2022

Hello @rydmike. Thank you for the detailed proposal. Labelling the request for the team to take it further.

@exaby73 exaby73 added a: text input Entering text in a text field or keyboard related problems framework flutter/packages/flutter repository. See also f: labels. f: material design flutter/packages/flutter/material repository. c: proposal A detailed proposal for a change to Flutter and removed in triage Presently being triaged by the triage team labels Oct 13, 2022
@rydmike
Copy link
Contributor Author

rydmike commented Oct 18, 2022

It is not really a proposal like the tag says.

There are actual discrepancies between Flutter Material 3 implementation and its web spec. Either Flutter implementation is incorrect or the Web spec is wrong, both can't be right. At least one of them should be corrected, which one I leave to the Flutter and Material team to determine. The diffs are documented in this issue.

Needed Flutter corrections are given as examples, assuming that the Web Material 3 spec is correct.

@exaby73
Copy link
Member

exaby73 commented Oct 18, 2022

Thanks @rydmike for the clarification. Updating the labels to reflect the issue better.

@exaby73 exaby73 added has reproducible steps The issue has been confirmed reproducible and is ready to work on found in release: 3.3 Found to occur in 3.3 found in release: 3.4 Found to occur in 3.4 and removed c: proposal A detailed proposal for a change to Flutter labels Oct 18, 2022
@darrenaustin
Copy link
Contributor

Thx @rydmike for the detailed bug report. @Hangyujin can you take a look at this?

@hannah-hyj hannah-hyj moved this to ✅ Done in Material 3 Oct 26, 2022
@exaby73 exaby73 added the r: fixed Issue is closed as already fixed in a newer version label Oct 26, 2022
@github-actions
Copy link

github-actions bot commented Nov 9, 2022

This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 9, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a: text input Entering text in a text field or keyboard related problems f: material design flutter/packages/flutter/material repository. found in release: 3.3 Found to occur in 3.3 found in release: 3.4 Found to occur in 3.4 framework flutter/packages/flutter repository. See also f: labels. has reproducible steps The issue has been confirmed reproducible and is ready to work on r: fixed Issue is closed as already fixed in a newer version
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants