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 missing var keyword #11

Merged
merged 1 commit into from
Aug 7, 2017
Merged

Fix missing var keyword #11

merged 1 commit into from
Aug 7, 2017

Conversation

Soreine
Copy link
Contributor

@Soreine Soreine commented Aug 5, 2017

The missing var keyword would crash when in Chrome, with an error "accented" is not defined

This would crash when used in Chrome for example.
Soreine added a commit to Soreine/hyper-gwent that referenced this pull request Aug 5, 2017
@Soreine
Copy link
Contributor Author

Soreine commented Aug 5, 2017

Well technically I think it fails whenever we use JS strict mode

zhouzi pushed a commit to Soreine/hyper-gwent that referenced this pull request Aug 6, 2017
* Added aliases

* Add failing test for accentuated letters

* Add back accents

* Support accents

* Fix error in Chrome

See tyxla/remove-accents#11
zhouzi pushed a commit to Soreine/hyper-gwent that referenced this pull request Aug 7, 2017
* Add Competitive_Gwent

* Refactor tooltip code

* Following mouse

* Keep card within viewport

* Added aliases

* Add failing test for accentuated letters

* Add back accents

* Support accents

* Fix error in Chrome

See tyxla/remove-accents#11

* Support GwentDB (don't double tooltips)

* Cleanup

* Wrote content for home page

* Wrote content for home page

* Add frog and fix ADC

* Hyper Gwent logo using Sketch

* Improve R

* Fix kerning

* Add colors 🌈

* Improving homepage style

* Add dandelion and geralt avatars

* Fix walker to skip script and styles tags

* Added emotes :p

* Fix responsiveness

* Another wordplay

* Dark theme

* Grammar

* Add link to issues

* Style issues

* Add gradient background

* Link to compose a message

* Add favicon

* Remove border from logo

* Add back border, but better

* Adjust button colors

* Add warning for non Chrome users. Fix rem

* Fix rem

* Remove list style

* Update manifest.json

* Limit extension to GwentDB decks

* Fix manifest.json

* Extend to GwentDB forums
Copy link
Owner

@tyxla tyxla left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@tyxla
Copy link
Owner

tyxla commented Aug 7, 2017

Thanks for that one!

@tyxla tyxla merged commit 3ed0bcb into tyxla:master Aug 7, 2017
@tyxla
Copy link
Owner

tyxla commented Aug 7, 2017

@Soreine If you want to use this one in your projects immediately, you can bump your dep version to 0.4.1.

@Soreine
Copy link
Contributor Author

Soreine commented Aug 7, 2017 via email

@Soreine Soreine deleted the patch-1 branch August 7, 2017 13:51
@tyxla
Copy link
Owner

tyxla commented Aug 7, 2017

I really appreciate your contribution 🙇

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.

2 participants