search

cortexrd / Knack-Toolkit-Library

Knack Toolkit Library
MIT License
26 stars 6 forks source link

Context-Sensitive Right-Click Menu Issue with User Filters #293

Closed chprojectsupport closed 5 months ago

chprojectsupport commented 5 months ago

Hi Guys,

Please see the below issue:

https://www.loom.com/share/2b77da778db54280adad889ffd895eea?sid=1ad52586-61b9-4ff9-ac04-68bfe9d769e0

cortexrd commented 5 months ago

Hi Carl,

I tried to reproduce this here but can't do it.

@CSWinnall, any luck on your side?

I suspect it might have something to do with the Live App Design settings. I tried a few changes like Legacy and Sticky Headers but no success.

Norm

cortexrd commented 5 months ago

Go it. It's this:

image

The z-index should be 6 not 3.

I will update this in next release.

@chprojectsupport, can you test the Beta version please?

Norm

chprojectsupport commented 5 months ago

Very strange, I've definitely had it working before on other apps.

I've re-checked and I don't have any other code running on the grid view, only the KTL sticky headers. If I remove _sth=, 700 it still renders behind the table header, with no sticky header?

Carl Holmes Director Mobile 07768 128128 www.chprojectsupportlimited.com

On Fri, 17 May 2024 at 14:21, Cortex R&D Inc. @.***> wrote:

Hi Carl,

I tried to reproduce this here but can't do it.

@CSWinnall https://github.com/CSWinnall, any luck on your side?

I suspect it might have something to do with the Live App Design settings. I tried a few changes like Legacy and Sticky Headers but no success.

Norm

— Reply to this email directly, view it on GitHub https://github.com/cortexrd/Knack-Toolkit-Library/issues/293#issuecomment-2117593238, or unsubscribe https://github.com/notifications/unsubscribe-auth/A7F6CSA43X2T3XKFYYE5ATTZCX74VAVCNFSM6AAAAABH4CNCXWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJXGU4TGMRTHA . You are receiving this because you authored the thread.Message ID: @.***>

cortexrd commented 5 months ago

And the Beta code doesn't solve that? ND

cortexrd commented 5 months ago

Can you inspect the header and check it's z-index value?

image

image

chprojectsupport commented 5 months ago

There isn't a z-index in the element.style section. I've also searched for "index" which produces no results in the DOM.

[image: image.png] Carl Holmes Director Mobile 07768 128128 www.chprojectsupportlimited.com

On Fri, 17 May 2024 at 14:37, Cortex R&D Inc. @.***> wrote:

Can you inspect the header and check it's z-index value?

image.png (view on web) https://github.com/cortexrd/Knack-Toolkit-Library/assets/42851359/90a295ed-d22e-41b2-9dbb-0c63987d19cb

image.png (view on web) https://github.com/cortexrd/Knack-Toolkit-Library/assets/42851359/6606869b-b96e-452a-ae6b-0d70b44b4a2e

— Reply to this email directly, view it on GitHub https://github.com/cortexrd/Knack-Toolkit-Library/issues/293#issuecomment-2117628098, or unsubscribe https://github.com/notifications/unsubscribe-auth/A7F6CSCGFMOTTCACVSKUGZTZCYB2LAVCNFSM6AAAAABH4CNCXWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJXGYZDQMBZHA . You are receiving this because you were mentioned.Message ID: @.***>

CSWinnall commented 5 months ago

Hi Norm

I just realised that we set the z-index of the menuDiv to 3 quite a long time ago. Because of this issue.

Craig

chprojectsupport commented 5 months ago

This has been resolved after talking with Craig this morning. There was some CSS setting the z-index to 20 which I have reduce to 2 and now the menu appears correctly. My bad 😢 Thank you Craig for all your help 🙏