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

bug: Messages SMS Zone Combobox width #6070

Closed
ophthal opened this issue Dec 31, 2022 · 2 comments · Fixed by #7307
Closed

bug: Messages SMS Zone Combobox width #6070

ophthal opened this issue Dec 31, 2022 · 2 comments · Fixed by #7307
Labels
Milestone

Comments

@ophthal
Copy link
Contributor

ophthal commented Dec 31, 2022

Describe the bug

IN /interface/main/messages.php

<span class="select2-selection select2-selection--single" role="combobox" aria-haspopup="true" aria-expanded="true" tabindex="0" aria-disabled="false" aria-labelledby="select2-SMS_patient-container" aria-owns="select2-SMS_patient-results"><span class="select2-selection__rendered" id="select2-SMS_patient-container" role="textbox" aria-readonly="true"></span><span class="" role="presentation"><b role="presentation"></b></span></span>

To Reproduce

This SMSM Box is only visible to MedEx users so here is a screen shot of how it appears in the OpenEMR Docker:latest

image

Expected behavior

image

need to add width:200px or something else to expand this "combobox" so it is visible.
I am not sure which class or css file can handle this but somewhere width:200px would work.

@robertdown
Copy link
Member

Just a note, we can't hard code a width in like this, it will break responsiveness.

@sjpadgett
Copy link
Sponsor Member

Need to give a default item to populate select.
Also for some ideas on config for select2 take a look at some ways I've implemented like in: portal/import_template_ui.php throughout.

@stephenwaite stephenwaite changed the title Messages: SMS Zone Combobox width bug: Messages SMS Zone Combobox width Apr 22, 2024
@adunsulag adunsulag added this to the 7.0.2.1 milestone Apr 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants