Skip to content
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

limel-dialog is shown cramped within the related-view #60

Closed
CarlitoSweden opened this issue Aug 23, 2018 · 4 comments
Closed

limel-dialog is shown cramped within the related-view #60

CarlitoSweden opened this issue Aug 23, 2018 · 4 comments
Assignees
Labels
design Discussion of new features

Comments

@CarlitoSweden
Copy link

Current behavior

limel-dialog is shown cramped in within the related-view and not in the whole browser window when opened.

Expected behavior

The limel-dialog should appear as it does when it's opened from a lime-object container modal style with backdrop

Environment

  • lime-elements version: 2.2.1

  • Framework used: StencilJS

  • Logs: Nothing more but the unrelated usual focus-trap error

@adrianschmidt adrianschmidt self-assigned this Aug 29, 2018
@adrianschmidt
Copy link
Collaborator

Hi!

Sorry for not responding earlier, I wasn't set as watching the whole repo, just mentions.

Are you using lime-elements in a plugin that you then use inside the webclient? If so, could you send me the plugin? Otherwise, if you could upload a screenshot or a gif of the behaviour, that would be very helpful 👍

@adrianschmidt adrianschmidt added question Further information is requested lime-elements and removed lime-elements labels Aug 29, 2018
@adrianschmidt
Copy link
Collaborator

AFK discussion sum-up:

The problem is that limel-dialog places the dialog element next to the trigger element in the DOM. This means that when shown in the related-view, the dialog can't overflow the confines of the related-view since any overflow is hidden.

This probably isn't anything we can fix in limel-dialog itself. I can see two different ways of fixing it:

  • The addonApp functionality that @jtennlo is working on will probably give us the fastest way to resolve this, since (I believe) it ought to be possible to put a dialog element in the addonApp component, and trigger it from a button in a different component.
  • Adding a dialog-service to lime-webclient. This should be a much more robust and much cleaner way to achieve "global" dialogs, but it will probably take a bit longer before we have this implemented.

@adrianschmidt adrianschmidt added the design Discussion of new features label Sep 3, 2018
@adrianschmidt
Copy link
Collaborator

The Lundalogik/crm-feature#549 is somewhat related to this. It doesn't explicitly discuss modal-type dialogs, but it discusses dialogs and toasts.

@adrianschmidt adrianschmidt removed the question Further information is requested label Sep 3, 2018
@adrianschmidt adrianschmidt changed the title limel-dialog is shown cramped in within the related-view limel-dialog is shown cramped within the related-view Oct 3, 2018
@adrianschmidt
Copy link
Collaborator

This is being addressed in Lundalogik/crm-feature#586

Closing this issue.

@ghost ghost removed the latent label Oct 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design Discussion of new features
Projects
None yet
Development

No branches or pull requests

3 participants