search

FlowFuse / flowfuse

Build bespoke, flexible, and resilient manufacturing low-code applications with FlowFuse and Node-RED
https://flowfuse.com
Other
265 stars 63 forks source link

Some operations are obscured by HS chat bubble widget #4488

Closed Steve-Mcl closed 1 day ago

Steve-Mcl commented 2 days ago

Nick has stated on Slack there is still pages in admin views that have this overlap issue. In particular team types.

See also: https://github.com/FlowFuse/flowfuse/pull/4477#issuecomment-2352563844

The CSS (while working locally) does not seem to affect the widget once deployed. Possibly due to how script injects the widget?

image

And despite the screenshots in the PR description, the iframe is still the large square - here I've used browser tools to draw its outline - nothing inside that square is clickable.

Originally posted by @Steve-Mcl in https://github.com/FlowFuse/flowfuse/issues/2897#issuecomment-2352673539

Steve-Mcl commented 2 days ago

FWIW, I finally got a reply from the MOD who anounched movable widgets.

Hi @SDMcLaughlin! The draggable widget feature is currently in beta but we plan to release it to all applicable HubSpot portals within the next month.

REF: https://community.hubspot.com/t5/HubSpot-Ideas/More-Chat-Widget-Styling-Options/idc-p/1040095/highlight/true#M193200

Steve-Mcl commented 2 days ago

Nick has stated on Slack there is still pages in admin views that have this overlap issue.

TeamTypes image

Steve-Mcl commented 1 day ago

verified on prod

users image

team types image

knolleary commented 14 hours ago

@Steve-Mcl To clarify - are we fixing this on a case-by-case basis for each and every view? Or are we fixing the highlevel layout to just work for all views?

I ask because https://app.flowfuse.com/admin/flow-blueprints is also affected

Steve-Mcl commented 13 hours ago

@Steve-Mcl To clarify - are we fixing this on a case-by-case basis for each and every view? Or are we fixing the highlevel layout to just work for all views?

@knolleary

No, not fixing at high level. Not all views use the same container. It was easier and quicker to approach each one as it is found.

Additionally, I have deliberately decorated each fix with the same searchable comment as it is my full intention to remove them all once the widget is draggable (which was the original request): https://github.com/FlowFuse/flowfuse/issues/2897#issuecomment-2299084550

I have been reliably informed that within approx 2 weeks time "the next month" (by a HS team member) that the drag feature will be out of beta - at which time it is my full intention to undo the hacks.

Steve-Mcl commented 13 hours ago

follow up issue here: https://github.com/FlowFuse/flowfuse/issues/4512