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

update README to reference the new hex widget #13

Merged
merged 5 commits into from May 10, 2021

Conversation

charlesdaniels
Copy link
Member

@charlesdaniels charlesdaniels commented May 8, 2021

Edit: this PR now also moves the new hex widget into the main widget package, per @andydotxyz.

@charlesdaniels
Copy link
Member Author

Looks like maybe go.mod needs updated, but I don't think that's related to this PR, since there were no code changes. Did app change upstream or something? Should I fix this as part of this PR, or does that need to be done separately.

The original hexwidget PR started before fyne-x really had any
established conventions, so I put it in it's own package. Since then, it
seems we've settled on keeping things all together in the widget
package.
Copy link
Member

@andydotxyz andydotxyz left a comment

Choose a reason for hiding this comment

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

Thanks for all this, but can the images be put into img/ please?

@charlesdaniels
Copy link
Member Author

Thanks for all this, but can the images be put into img/ please?

Done in b67a3c2.

README.md Outdated Show resolved Hide resolved
@charlesdaniels charlesdaniels merged commit a37ac4f into fyne-io:master May 10, 2021
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

3 participants