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

Different user's level popup is overlapping together #152

Closed
MrOrz opened this issue Feb 10, 2019 · 2 comments
Closed

Different user's level popup is overlapping together #152

MrOrz opened this issue Feb 10, 2019 · 2 comments

Comments

@MrOrz
Copy link
Member

MrOrz commented Feb 10, 2019

Despite the fact that cofacts/rumors-api#95 is fixed by @nonumpa 's cofacts/rumors-api#123, the user level display is still incorrect.

overlapping

Seems that all user's level popups are all overlapping together.

Example: https://cofacts.g0v.tw/article/3qlrkscnnx950

It's a good first issue I think.

nonumpa added a commit that referenced this issue Feb 24, 2019
use balloon-css instead of react-tooltip
@nonumpa
Copy link
Member

nonumpa commented Feb 24, 2019

According to the meeting note : use balloon.css instead.

I use npm install balloon-css, but don't know how to import it correctly.

import 'balloon-css' is not work, but import '../node_modules/balloon-css/balloon.css' seems not good.

Here is the commit.

@MrOrz MrOrz closed this as completed in 6711114 Feb 27, 2019
MrOrz added a commit that referenced this issue Feb 27, 2019
@MrOrz
Copy link
Member Author

MrOrz commented Feb 27, 2019

#153 fixes this.

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

No branches or pull requests

2 participants