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

Use flags-css-icon submodule #71

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

pedro-mendonca
Copy link
Contributor

For comparing purposes, I'm making this pull request.
Please consider this a full working suggestion only, feel free to comment or deleting it :)
You must know that this is made on top of your last commit at this time, and should be all checked.
Also consider the todo list and considerations list in #52

Make alt texts translatable
Changed flags from fullname to code name 'xx.png' to 'xx' to use it as sufix code in css classes
Added import of /flag-icon-css/css/flag-icon.min.css
Change flags path to flag-icon-css submodule
Uncomment and re-add function qtranxf_add_css() to allow qtranslate.css loading
Changes:
Added suffix .svg in background image loading script of admin main language
Added suffix .svg in background image loading script of admin dropdown list of languages
Uncomment and re-add qtranxf_add_css()
Added 'svg' to file list in (jpeg|jpg|gif|png|svg)
Changed options values in flag selector
Corrected javascript to add .svg in language_flag switcher
Changed IMG to SPAN new flags in list of enabled languages (NOW WITHOUT ALT TEXT** need to check)
Changed IMG to SPAN new flags in table of available languages (NOW WITHOUT ALT TEXT** need to check)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants