Skip to content

Conversation

@gspencergoog
Copy link
Contributor

This renames the chip's border attribute to 'shape' so that it is consistent with the other places where we supply a ShapeBorder.

This is a breaking change, but a very small one: the border attribute was introduced only a few months ago, and is not widely used.

@gspencergoog gspencergoog requested a review from Hixie April 5, 2018 16:52
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

extraneous blank lines?

@Hixie
Copy link
Contributor

Hixie commented Apr 5, 2018

LGTM

@gspencergoog gspencergoog merged commit 6055c56 into flutter:master Apr 5, 2018
@gspencergoog gspencergoog deleted the rename_border branch April 17, 2018 03:26
DaveShuckerow pushed a commit to DaveShuckerow/flutter that referenced this pull request May 14, 2018
This renames the chip's border attribute to 'shape' so that it is consistent with the other places where we supply a ShapeBorder.

This is a breaking change, but a very small one: the border attribute was introduced only a few months ago, and is not widely used.
teriyakijack added a commit to teriyakijack/flutter that referenced this pull request Jul 17, 2018
* 'master' of https://github.com/flutter/flutter:
  Roll engine to aa9ce70 (flutter#16289)
  Rename chip's border attribute to shape for consistency. (flutter#16276)
  Added BeveledRectangleBorder ShapeBorder (flutter#16279)
  Roll engine to version be07059 (flutter#16264)
  Make Podfiles work with Cocoapods 1.5.0 (flutter#16273)
  Updated appearance of filled TextFields - added UnderlineInputBorder.borderRadius (flutter#16272)
  Handle whitespace in entry values in the AAPT badging parser (flutter#16245)
  a11y adjustments for the Bottom app bar demo (flutter#16238)
  Play butterfly video from asset instead of network (flutter#16269)
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants