Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix fragments documentation #201

Merged
merged 1 commit into from
Aug 24, 2021
Merged

Fix fragments documentation #201

merged 1 commit into from
Aug 24, 2021

Conversation

CalvinWalzel
Copy link
Contributor

Description

Basic syntax -> Fragments currently renders the fragment tag invalid. I think this is also the bug mentioned in #199. The issue was a double space between the def and the method name.

How Has This Been Tested?

This has been tested manually in the browser. See images below for comparison.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist:

  • I have read the CONTRIBUTING document.
  • I have updated/added documentation affected by my changes.
  • I have added tests to cover my changes.

Images

Before
CleanShot 2021-08-24 at 22 25 47@2x
After
CleanShot 2021-08-24 at 22 29 49@2x

Fixes syntax for fragments example in documentation
@CalvinWalzel CalvinWalzel changed the title Remove double space Fix fragments documentation Aug 24, 2021
@somebee somebee merged commit 6ad4e9a into imba:master Aug 24, 2021
@CalvinWalzel CalvinWalzel deleted the fix-fragments-syntax branch August 24, 2021 20:35
@somebee
Copy link
Member

somebee commented Aug 24, 2021

Thanks :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants