Skip to content

Add an instruction page for adding iOS App Clip - #4592

Merged
sfshaza2 merged 2 commits into
flutter:masterfrom
xster:app-clip
Sep 24, 2020
Merged

sfshaza2 merged 2 commits into
flutter:masterfrom
xster:app-clip

Conversation

@xster

@xster xster commented Sep 9, 2020 •

Copy link
Copy Markdown
Member

Fixes flutter/flutter#60029

The PNGs are opti-png'ed.

@xster
xster requested a review from sfshaza2 September 9, 2020 09:16
@googlebot googlebot added the cla: yes Contributor has signed the Contributor License Agreement label Sep 9, 2020
@xster
xster requested a review from redbrogdon September 9, 2020 09:29
Comment thread src/docs/development/platform-integration/ios-app-clip.md
Comment thread src/docs/development/platform-integration/ios-app-clip.md Outdated
Comment thread src/docs/development/platform-integration/ios-app-clip.md Outdated
in the future.
{{site.alert.end}}

This guide describes how to manually add another Flutter-rendering iOS App Clip

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.

Another? Is one already there by default?

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.

Ah sorry, it's not clear. There is another Flutter-rendering target already by default (their main app target).

Comment thread src/docs/development/platform-integration/ios-app-clip.md Outdated
Comment thread src/docs/development/platform-integration/ios-app-clip.md Outdated
Comment thread src/docs/development/platform-integration/ios-app-clip.md Outdated
Comment thread src/docs/development/platform-integration/ios-app-clip.md Outdated
Comment thread src/docs/development/platform-integration/ios-app-clip.md Outdated
Comment thread src/docs/development/platform-integration/ios-app-clip.md Outdated
Comment thread src/docs/development/platform-integration/ios-app-clip.md Outdated
@xster

xster commented Sep 9, 2020

Copy link
Copy Markdown
Member Author

Thanks Shams for the detailed review again! I did most of what you asked. Regarding the space between the headers, I was looking at the page again and thought all the headers and subheaders were a bit too obnoxious and created too much big text and space around the actual content. I made everything a bit denser with just bold text. Let me know if it's ok.

@sfshaza2 sfshaza2 left a comment

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.

This is looking good! Staged here: https://sz-flutter.firebaseapp.com/docs/development/platform-integration/ios-app-clip

Thanks for optimizing the images!

The final bit that needs fixing, is that the link to the samples app is a 404. Once that's fixed, I can land this.

Comment thread src/docs/development/platform-integration/ios-app-clip.md
@xster

xster commented Sep 10, 2020

Copy link
Copy Markdown
Member Author

Sounds good. Thanks for the review!

I don't have the sample app yet unfortunately 😅

I'll bug Andrew tomorrow.

@sfshaza2

sfshaza2 commented Sep 10, 2020 via email

Copy link
Copy Markdown
Contributor

@redbrogdon

Copy link
Copy Markdown
Contributor

SG! Andrew needs a little bugging. ;)

Let the record show that I was bugged forthwith by @xster.

@jmagman

jmagman commented Sep 21, 2020

Copy link
Copy Markdown
Member

Can we remove the sample reference and update this later when we have it? The instructions are still useful without the sample app.

@xster

xster commented Sep 23, 2020

Copy link
Copy Markdown
Member Author

The samples are in. Should be ready now.

@xster
xster requested a review from sfshaza2 September 24, 2020 05:41

@sfshaza2 sfshaza2 left a comment

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.

LGTM

@sfshaza2
sfshaza2 merged commit 55db234 into flutter:master Sep 24, 2020
filiph pushed a commit to filiph/website that referenced this pull request Feb 22, 2021
* Add an instruction page for adding iOS App Clip

* review
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Contributor has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support app clips on iOS 14

5 participants