Floating detail panel

[bryan-brancotte]

Checklist

• I indicated which issue (if any) is closed with this PR using closing keywords
• I only changed lines related to my PR (no bulk reformatting)
• I indicated the source and check the license if I re-use code, or I did not re-use code
• I made my best to solve only one issue in this PR, or explain why multi had to be solved at once.

Issue

close/fix/resolve #167

details

[raashika03]

This looks good @bryan-brancotte exactly like that photoshopped one.
width of the tooltip box should be little more for better visualisation. cause a lot of space is there so.

[bryan-brancotte]

[raashika03]

A small remark here. Details table content flows for some screen sizes. Can you have a look over the same.
This might be helpful @bryan-brancotte

[raashika03]

I didn't understand are you planning to remove term details panel in the right side, completely from the main page by displaying on hovering.

There's no point of displaying it when it's already there on hovering.

May be only for devices which doesn't have hovering action we can let the term details panel as it's. What do you think?
Moreover this way we can reduce CPU consumption as you told here.

I'm pointing this out cause when details table is displayed on hovering I didn't look over the right-side panel even once. Didn't need that. So, it might be removed now.

And yeah those settings ideas are really great 👏

But @hmenager @matuskalas can tell better if they still want that table right-side. I'm not yet so much acquainted with the usability of edam-browser 😅

[raashika03]

How about linking cookies for the storage of the state chosen for these three new buttons. I find that I've to change every time after refreshing. @bryan-brancotte I find this useful. 😅

This can be covered in other PR, cause current is itself a big jump in UI.

[raashika03]

@bryan-brancotte do you think this should also be fixed.
Placement of tooltip can be optimised.
PS- scroll-bar is generated but that's not useful. Scrolling isn't working here.

[bryan-brancotte]

I didn't understand are you planning to remove term details panel in the right side, completely from the main page by displaying on hovering.

No I don't, I even don't like to have the tooltip when not in fullscreen (that is why i disabled it years ago)

There's no point of displaying it when it's already there on hovering.

agrees, but some might find usefull to hover with only community usage, (maybe ?)

How about linking cookies for the storage of the state chosen for these three new buttons. I find that I've to change every time after refreshing. @bryan-brancotte I find this useful. sweat_smile

I don't really like the tooltip, so I hesitated to save it in storage, for now no, but indeed it would be sound to have it in storage.

This can be covered in other PR, cause current is itself a big jump in UI.

Tooltip causes UI jump/glitch that is why I want to reduce its usage when not in fullscreen.

Placement of tooltip can be optimised.

Clearly, but it would be by hand, not a "simple" bootstrap attribut.

PS- scroll-bar is generated but that's not useful. Scrolling isn't working here.

It works sometime, but not all the time so not reliable

[raashika03]

There's no point of displaying it when it's already there on hovering.

agrees, but some might find usefull to hover with only community usage, (maybe ?)

Yes it's a good feature. It'll be helpful for sure. Hovering is quick.

I don't really like the tooltip, so I hesitated to save it in storage, for now no, but indeed it would be sound to have it in storage.

Tooltip causes UI jump/glitch that is why I want to reduce its usage when not in fullscreen.

Placement of tooltip can be optimised.

Clearly, but it would be by hand, not a "simple" bootstrap attribut.

Ok @bryan-brancotte now everything sounds good to me.
Let all these for now then in future those cookies, & Ui glitch can be looked upon as per the user feedback. Good to go now 🎉