Skip to content

[Discuss] A NotificationBuilder - #18136

Closed
xster wants to merge 1 commit into
flutter:masterfrom
xster:notification-builder
Closed

xster wants to merge 1 commit into
flutter:masterfrom
xster:notification-builder

Conversation

@xster

@xster xster commented Jun 2, 2018

Copy link
Copy Markdown
Member

Not a real PR. Just thinking out loud. I think something like this could be useful. Worth adding to the framework?

@xster
xster force-pushed the notification-builder branch from bbbbdb9 to 8a7e86b Compare June 2, 2018 03:59
@Hixie

Hixie commented Jun 2, 2018

Copy link
Copy Markdown
Contributor

we should wait until we have a real need before adding it, but the idea seems sound

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

maybe a list? there could be many notifications.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

what does it mean concretely? Does scrollables send multiple notifications per frame?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

They can, sure. There's nothing about the notification system that limits notifications to one per frame.

@xster

xster commented Jun 3, 2018

Copy link
Copy Markdown
Member Author

I was hoping to use something like this in a demo for my presentation to show widget composability :)

@Hixie

Hixie commented Jun 4, 2018

Copy link
Copy Markdown
Contributor

What is the use case in your presentation?

@xster

xster commented Jun 5, 2018

Copy link
Copy Markdown
Member Author

I'm trying to show that the 'middleware' in the framework are all customizable by making a ScrollView also rotate when scrolling :)

@xster
xster force-pushed the notification-builder branch from 8a7e86b to 69553c5 Compare June 7, 2018 20:31
@xster
xster force-pushed the notification-builder branch from 69553c5 to 8615c24 Compare June 9, 2018 06:52
@Hixie

Hixie commented Jul 3, 2018

Copy link
Copy Markdown
Contributor

@xster should we close this until we have a need for it?

@xster

xster commented Jul 3, 2018

Copy link
Copy Markdown
Member Author

Sure

@xster xster closed this Jul 3, 2018
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 10, 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