Skip to content

Add tooltips - #9

Merged
emareg merged 7 commits into
emareg:masterfrom
TimDaub:feat/tooltips
Jun 1, 2021
Merged

Add tooltips#9
emareg merged 7 commits into
emareg:masterfrom
TimDaub:feat/tooltips

Conversation

@TimDaub

@TimDaub TimDaub commented May 11, 2021

Copy link
Copy Markdown
Contributor

FYI: Note, I had to remove overflow-y: hidden and overflow-x from your reset script to make the tooltips work in tables (see 8f1df42). I tried breaking out without removing those two lines, but I failed.

@TimDaub

TimDaub commented May 12, 2021

Copy link
Copy Markdown
Contributor Author

I noticed that removing the overflow-y and overflow-x values, the responsive tables break :/

I intensely searched online for a solution to pop out of the table's hidden overflow, but without JS it seems not doable. Any ideas?

TimDaub added 2 commits May 17, 2021 13:57
- Remove four direction positions of tooltip
@TimDaub

TimDaub commented May 17, 2021

Copy link
Copy Markdown
Contributor Author

Updates:

@emareg
emareg merged commit 89aa8c8 into emareg:master Jun 1, 2021
@emareg

emareg commented Jun 1, 2021

Copy link
Copy Markdown
Owner

Thanks, I have merged your changes but I I think I will update the tooltips again later. I have an idea to combine these with the citation popup.

@TimDaub

TimDaub commented Jun 1, 2021

Copy link
Copy Markdown
Contributor Author

great thanks. And I'm looking forward to your improved version

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.

Interested in classless tooltips?

2 participants