Skip to content

Gallery about box: state Flutter's status and link to our website - #4746

Merged
HansMuller merged 3 commits into
flutter:masterfrom
HansMuller:gallery_about_box
Jun 25, 2016
Merged

HansMuller merged 3 commits into
flutter:masterfrom
HansMuller:gallery_about_box

Conversation

@HansMuller

@HansMuller HansMuller commented Jun 24, 2016 •

Copy link
Copy Markdown
Contributor

Fixes #4512
Fixes #4584

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.

I'm surprised you're using a shortened link instead of just https://github.com/flutter/flutter/tree/master/examples/flutter_gallery. Does the shortened link allow you to change it after the fact or some other value-add?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I made it short because the long link looks terrible. A side benefit is that it gool.gl reports usage. If it's possible to change the link's value, it's not obvious (to me) from the goo.gl UI.

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.

Oh. I see. You're showing the URL itself in the dialog.

On Friday, June 24, 2016, Hans Muller [email protected] wrote:

In examples/flutter_gallery/lib/gallery/drawer.dart
#4746 (comment):

  •                    "developers build high-performance, high-fidelity, mobile "
    
  •                    "apps for iOS and Android from a single codebase. This "
    
  •                    "gallery is a preview of Flutter's many widgets, behaviors, "
    
  •                    "animations, layouts, and more. Learn more about Flutter at "
    
  •                  ),
    
  •                  new LinkTextSpan(
    
  •                    style: aboutLinkStyle,
    
  •                    url: 'https://flutter.io'
    
  •                  ),
    
  •                  new TextSpan(
    
  •                    style: aboutTextStyle,
    
  •                    text: ".\n\nTo see the source code for this app, please visit "
    
  •                  ),
    
  •                  new LinkTextSpan(
    
  •                    style: aboutLinkStyle,
    
  •                    url: 'https://goo.gl/iv1p4G'
    

I made it short because the long link looks terrible. A side benefit is
that it gool.gl reports usage. If it's possible to change the link's
value, it's not obvious (to me) from the goo.gl UI.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/pull/4746/files/3b77b85fe28882430d07f8f985e86a71a64ba13d#r68470524,
or mute the thread
https://github.com/notifications/unsubscribe/ALTvi6il2KRa8mH1ebNWgiECXL4pckFmks5qPFaRgaJpZM4I-KbY
.

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.

We could make a short link on flutter.io itself. It'd be prettier.

@Hixie

Hixie commented Jun 24, 2016

Copy link
Copy Markdown
Contributor

LGTM

@HansMuller
HansMuller merged commit 0a4f96e into flutter:master Jun 25, 2016
@HansMuller
HansMuller deleted the gallery_about_box branch June 25, 2016 00:54
@HansMuller HansMuller mentioned this pull request Jun 27, 2016
cdotstout pushed a commit to cdotstout/flutter that referenced this pull request Apr 3, 2018
…oid (flutter#4746)

* deprecate snapshot_override, which is an obsolete predecessor of hot reload
* give the APKAssetProvider to the engine in the initial call to RunBundle
* later calls to Engine::RunBundleAndSource or Engine::SetAssetBundlePath
  will replace the APK asset provider with a DirectoryAssetBundle that uses
  the newly pushed assets
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 15, 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.

Add an example to Gallery of using new open url service Gallery: State Flutter's status and link to our website

3 participants