-
-
Notifications
You must be signed in to change notification settings - Fork 176
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
Language Chooser dropdown keeps style across different OS #316
Comments
Some problem with the markdown |
Hello @fabiocaccamo ! When I wrote the issue 314 I also realized that we could set the dropdown CSS style to keep the same design across different O.S. Since I was trying to contribute to you and make an improved style on that component for my current projects I discovered that is a native component. It means the style depends on the OS. |
So I decided to develop a dropdown based on this sample to combine solutions to render always the same design. |
I believe it could be improved by the team and used on new releases... Sharing codes../templates/language_chooser.html
../css/extra_base_site.css
../js/extra_base_site.js
|
Hi @saymoncoppi, Anyway, this is a good idea and I'm open to support a new language chooser control style: "Dropdown". ![]() PS. In the CSS, the theme CSS variables should be used: |
Hello @fabiocaccamo honestly I don't know which is the next step to take. Is it ok for you if we schedule a call to talk about it? Maybe you could teach me what should I do... feel free to call me any time |
The next step is making a fork of this repo, cloning it locally, adding an option to https://github.com/fabiocaccamo/django-admin-interface/blob/main/admin_interface/models.py#L138 and adding the markup to https://github.com/fabiocaccamo/django-admin-interface/blob/main/admin_interface/templates/admin_interface/language_chooser.html |
Here you can find also a step by step guide: |
@saymoncoppi I close this issue because it's not my intention to implement a custom dropdown. |
No description provided.
The text was updated successfully, but these errors were encountered: