Skip to content
This repository has been archived by the owner on Mar 6, 2024. It is now read-only.

Docs for widgets #140

Merged
merged 5 commits into from Jul 9, 2021
Merged

Docs for widgets #140

merged 5 commits into from Jul 9, 2021

Conversation

jpnurmi
Copy link
Contributor

@jpnurmi jpnurmi commented Jul 8, 2021

Added missing docs for:

  • DropdownBuilder
  • MenuButtonBuilder
  • LocalizedView
  • OptionCard

I didn't bother with AffinityExpansionTile because it's going to be replaced by ExpansionTile.controlAffinity.

@codecov
Copy link

codecov bot commented Jul 8, 2021

Codecov Report

Merging #140 (b93fe0c) into main (ecda535) will not change coverage.
The diff coverage is 100.00%.

❗ Current head b93fe0c differs from pull request most recent head 8f76ddd. Consider uploading reports for the commit 8f76ddd to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##             main     #140   +/-   ##
=======================================
  Coverage   70.87%   70.87%           
=======================================
  Files          27       27           
  Lines        1102     1102           
=======================================
  Hits          781      781           
  Misses        321      321           
Impacted Files Coverage Δ
...esktop_installer/lib/widgets/dropdown_builder.dart 100.00% <ø> (ø)
..._desktop_installer/lib/widgets/localized_view.dart 100.00% <ø> (ø)
...top_installer/lib/widgets/menu_button_builder.dart 100.00% <ø> (ø)
...ntu_desktop_installer/lib/widgets/option_card.dart 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ecda535...8f76ddd. Read the comment docs.

@oSoMoN
Copy link
Contributor

oSoMoN commented Jul 9, 2021

LGTM, thanks!

@oSoMoN oSoMoN merged commit a76fcd6 into canonical:main Jul 9, 2021
@jpnurmi jpnurmi deleted the widget-docs branch July 9, 2021 16:12
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.

None yet

2 participants