arc2qgis / qlyrx

GNU General Public License v2.0
15 stars 5 forks source link

Save style to QML asks to choose a file not a folder #11

Closed rodreras closed 3 years ago

rodreras commented 3 years ago

Hey! I just saw you updated the plugin so I ran to take a look. I think I found a little issue.

When I click the option Save Style to QML, it opens a saving file window. It asks me to choose a .qml file, not to save as this extension. See the printscreens below:

Opening the plugin window:

Captura de tela de 2021-10-14 10-40-30

Browsing files:

Captura de tela de 2021-10-14 10-40-50

Is that really supposed to happen or is it a bug?

bogind commented 3 years ago

Hi, Yes, the intended behavior is indeed file selection and not folder selection. We use the QGIS QgsFileWidget for this, but you should be able to enter the name of a new file. What operating system are you using? This seems like an issue with the file explorer QGIS is opening.

rodreras commented 3 years ago

I'm currently using Ubuntu 20.04 and QGIS 3.16.11.

Ok, @bogind I misunderstood I was supposed to put the name of a new file. Sorry for that.

I'd like to use this same space to better understand how the plugin works.

I have a .lyrx file with several symbols for lithological classification.

After choosing the option "Load style from QML/SLD", it rises this error.

Captura de tela de 2021-10-17 13-11-11

Python Version: 3.8.10 (default, Sep 28 2021, 16:10:42) [GCC 9.3.0] QGIS Version: 3.16.11-Hannover Hannover, 26cc1c762f

This Python error might be because of my QML file or a problem in code?

bogind commented 3 years ago

I'm currently using Ubuntu 20.04 and QGIS 3.16.11.

Ok, @bogind I misunderstood I was supposed to put the name of a new file. Sorry for that.

I'd like to use this same space to better understand how the plugin works.

I have a .lyrx file with several symbols for lithological classification.

After choosing the option "Load style from QML/SLD", it rises this error.

Captura de tela de 2021-10-17 13-11-11

Python Version: 3.8.10 (default, Sep 28 2021, 16:10:42) [GCC 9.3.0] QGIS Version: 3.16.11-Hannover Hannover, 26cc1c762f

This Python error might be because of my QML file or a problem in code?

"Load style from QML/SLD" is a different tool, it's meant for easy and fast loading of styles already in SLD\QML from your file system. It wouldn't work with .lyrx files.

You seem to have found a bug in that tool, which should be now fixed with v0.3.3.

You can download it from releases and install from zip, I'll upload it to the plugin repository and hopefully it will be updated in the a few days.

regarding your original issue, this seems to be an issue with the QgsFileWidget on ubuntu, so I recommend filing an issue on the QGIS main repository. Closing this issue now.

rodreras commented 3 years ago

Thanks for you support!

And I'd like to make a disclamer: the last message I wrote, I was supposed to say .qml file intead of .lyrx. It was a lack of attention. I really appreciate your support.

If there's any translating program for you plugin, I can help with PT-BR.

Thanks!

bogind commented 3 years ago

@rodreras We don't currently use the translation feature, mostly because we don't have a lot of text, But I'll add an enhancement issue about this and might get around to adding it in a future release.