-
Notifications
You must be signed in to change notification settings - Fork 21
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
Support for custom popup actions using browser mod integration #204
Support for custom popup actions using browser mod integration #204
Conversation
Hello, This doesn't work for me. does it work on your non-dev version ? Calling popup via service instead of fire-dom works (but on all browsers instead of the one clicking, as expected with serverwide popup service event) so the popups do work. how im calling it right now: (also tested without the style part)
|
Yes, it does work here on all my devices, even on the mobile companion. I assume you:
If browser_mod is not active somehow, firing the On the other hand, the service call worked at that moment, ie browser_mod was installed as a known service, but was unable to receive events! Does browser_mod display an icon in the lower right corner of the screen after ctrl-F5? |
yes, enabled, auto-reg enabled, im just not able to get it to work on SAK card. i indeed get absolutely nothing.
|
Did you test an empty card with a circle/rectex and the browser_mod popup? I know from testing that browser_mod is a bit finicky on which HTML/SVG Element the event is fired. If fired on the "wrong" element (whatever that is), no popup is displayed, and nothing is output to the browser console. I tested with 5 cards and tools. Testing all variations with SAK is a bit difficult. |
just tested this and does not work either:
|
After a restart I got this error in the browser console: "Service parameter not specified in service call": Did not change a single thing of course. Will check the sources of browser_mod to see what this is about... The data part is empty. That is very weird of course if the view/card is not changed... Cleared the browser cache, and now everything works again 😯 NTS: A direct service call from within SAK does the same using the spread operator. But did not clone the action config for the Change will be a direct commit and part of v2.4.4 |
If you can tell me how i can check my console/logs im happy to check what it says over here, or is that no longer necessary seeing your edit about 2.4.4. |
I will release v2.4.4 with the change in how the eventdata is passed. You could wait for that. But checking the console output of Chrome is very easy:
If the console is not enabled, I thought that the Esc key should enable the console. As you see, most cards display a startup message with their version. |
That explains why popup's are not working for you! That might be the browser cache. You can disable that. Next to the console window you see 'network' in the tab. |
Yep, That did the trick! I even always refresh with CTRL+R which used to be a hard refresh which i thought also cleared the cache.. apparantly not then.. |
Great!
Yep, that sometimes works, and in many cases, it doesn't. So I always have that cache disabled if I'm developing. You can get fairly crazy if you think you fixed something, and it doesn't work due to the cache not being cleared. Grrrrrr... Have fun with the pop-ups 😄 |
Cleared front-end cache on app and also fixed for mobile app! |
Thanks! now i dont have server-wide popups anymore on my SAK cards :D |
Great! I will put that in the release notes: Don't forget to clear the caches! |
Description
By supporting the 'fire-dom-event' with
ll-custom
type, browser_mod is supported which allows several extensions like notifications, popups, etc.Related Issue
Motivation and Context
How Has This Been Tested?
Tested with
sak_layout_fce_light_with_slider2.yaml
test card:Screenshots (if appropriate):