Skip to content

[dash] Updates for widgets pages#1459

Merged
fertolg merged 2 commits into
dashfrom
widgets-catalog
Oct 15, 2018
Merged

[dash] Updates for widgets pages#1459
fertolg merged 2 commits into
dashfrom
widgets-catalog

Conversation

@fertolg

@fertolg fertolg commented Oct 11, 2018

Copy link
Copy Markdown
Contributor

@fertolg fertolg requested a review from chalin October 11, 2018 23:17
@googlebot googlebot added the cla: yes Contributor has signed the Contributor License Agreement label Oct 11, 2018
Comment thread src/development/ui/widgets.md Outdated
<a class="card" href="/development/ui/widgets/{{section.id}}">
<div class="card-body">
<header class="card-title">{{section.name}}</header>
<p class="card-text">{{section.description | escape}}</p>

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.

If I don't escape here, one of the descriptions contains html (a link), which should be fine, but for some reason when it outputs it doesn't seem to be outputting correctly and it totally messes up the second card.

@fertolg fertolg force-pushed the widgets-catalog branch 2 times, most recently from b3f96ec to 98b8cc2 Compare October 12, 2018 22:57
@fertolg fertolg changed the title WIP [dash] Updates for widgets pages Oct 12, 2018
@Sfshaza

Sfshaza commented Oct 15, 2018

Copy link
Copy Markdown
Contributor

@fertolg, this seems to be failing because it can't find the widgets. They are currently in /api-and-reference/widgets.

- adds bootstrap cards for widget catalog and the widget index
- removes “flutter by google” nav logo
- reorganizes styles to keep cards more consistent across different pages (showcase, widgets, etc).
@fertolg

fertolg commented Oct 15, 2018

Copy link
Copy Markdown
Contributor Author

Thanks @Sfshaza ! Fixed the broken links and the build is now green. I'll submit this PR so that I can move along with other things.

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.

3 participants