<div dir="ltr">On Mon, Jun 5, 2017 at 6:32 PM, Stéphane Henriod <span dir="ltr"><<a href="mailto:stephanebek@gmail.com" target="_blank">stephanebek@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Alessandro<div><br></div><div>replies <font color="#ff0000">inline</font><span><br><br>Le lundi 5 juin 2017, Alessandro Pasotti <<a href="mailto:apasotti@gmail.com" target="_blank">apasotti@gmail.com</a>> a écrit :<br></span><span><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div><div class="gmail_extra"><div class="gmail_quote">On Mon, Jun 5, 2017 at 1:54 PM, Stéphane Henriod <span dir="ltr"><<a>stephanebek@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Alessandro<div><br></div><div><div>First of all, thank you so much for this work, it looks really great already!</div><div><br></div><div>Here a few comments / ideas after a few quick tests, I let you filter which ones could actually be part of this phase and which ones should be left out for a future phase 2</div></div></blockquote><div><br><div>Hi, <br><br></div>thanks for testing!<br></div></div></div></div></div></div></blockquote><div><br></div></span><div><font color="#ff0000">That's the least I can do when someone has already done the hard part, i.e. developing :-)</font></div><span><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div><div class="gmail_extra"><div class="gmail_quote"><div><br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div><br></div><div>- "Raster" is not consistent: it is confusing that the file selector opens directly. The behavior should mimick the "vector" (open the file selector only when the user clicks on "browse")</div></div></blockquote><div><br></div><div>yeah, I know,  this has been already discussed in the PR comments, please have a look.<br></div></div></div></div></div></div></blockquote><div><br></div></span><div><font color="#ff0000">Yes, I've seen that. But I was not sure  if a consensus has been reached. Will it stay like this for now or is it still open for discussion? I would favor the "extra-click" that would make it more consistent with the other providers</font></div><span><div><font color="#ff0000"><br></font></div><div><font color="#ff0000"><br></font></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div><div class="gmail_extra"><div class="gmail_quote"><div></div></div></div></div></div></div></blockquote></span></div></blockquote><div><br></div><div>Nothing is carved in stones and I don't know if a consensus has been reached (I even don't know how to judge whether consensus has been reached), but I feel the other way: I prefer efficiency to consistency in this particular case. But maybe the strongest reason to not spend much time on it is that it's a temporary solution: adding a raster (or a vector file) from the filesystem is really something that can be done right now from the browser, but this is all in the PR discussion.<br></div><div><br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><span><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div><div class="gmail_extra"><div class="gmail_quote"><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div><br></div><div>- The "DB2" logo should be cleaned (at least add transparency)</div></div></blockquote><div><br></div><div><br>Sorry, I've no graphics design skills, and this task is really something that a not-programmer can contribute to. Changing the icons was not in scope btw.<br></div></div></div></div></div></div></blockquote><div><br></div><div><br></div></span><div><font color="#ff0000">I don't have such skills either but I'd be happy to try and at least make the background transparent. I however don't know where to find the original icon and how to push it back, once I have (maybe) successfully improved it. (This had also been raised by Nyall)</font></div><span><div><br></div><div><br></div></span></div></blockquote><div><br></div><div>You can find icons here: <a href="https://github.com/qgis/QGIS/tree/master/images/themes/default" target="_blank">https://github.com/qgis/QGIS/<wbr>tree/master/images/themes/<wbr>default</a><br><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><span><div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div><div class="gmail_extra"><div class="gmail_quote"><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div><br></div><div>- When I click on "Browse", the file selector window opens behind the Data Source Manager</div></div></blockquote><div><br></div><div>Not sure I understand what you mean here, probably need more details, and it seems like a bug.<br></div></div></div></div></div></div></blockquote><div><br></div></span><div><font color="#ff0000">Here what I mean: <a href="https://vimeo.com/220330186" target="_blank">https://vimeo.com/220330<wbr>186</a> </font></div><div><font color="#ff0000"><br></font></div><div><font color="#ff0000">But it is not super consistent and doesn't necessarily happen everytime</font></div><div><font color="#ff0000"><br></font></div><div><font color="#ff0000">I opened an issue here: <a href="https://issues.qgis.org/issues/16666" target="_blank">https://issues.qgis.org/<wbr>issues/16666</a></font></div></div></blockquote><div><br></div><div><br>Thanks, I'll definitely have a look, you are using the dialog in modeless mode, aren't you?<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><span><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div><div class="gmail_extra"><div class="gmail_quote"><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div><br></div><div>- I would change the order of the providers: move the "Delimited text" just below raster (so that we have all file-based providers together). And maybe separate (visually) the providers in sections: "Files", "Databases", "Web services", "Virtual" (or something like this). Also,"ArcGIS Feature Server" should probably be just after "ArcGIS Map Server"</div></div></blockquote><div><br></div><div><br>If I'm not mistaken, the order is  the same order we had before for the menu items and the icons, I did not alter the order (at least that was not my intention).<br></div></div></div></div></div></div></blockquote><div><br></div></span><div><font color="#ff0000">I indeed think that you kept the original order. But would it be out of the scope of this first phase to clean up and make this order more intuitive? It seems to me like a good opportunity but maybe I am underestimating the associated workload?</font></div><span><div><br></div></span></div></blockquote><div><br></div><div>No, it's not a big workload, but changing the order wasn't in my plans. Feel free to make a proposal and a PR and let's see what other people thinks (I've not a strong opinion on this topic).<br></div><div></div><div> <br></div><div>As soon as I find some time to work on this again I will focus on a small refactoring to move the classes from app to gui (that has no impact at all on UI/UX).<br></div><div><br clear="all"></div></div>Cheers.<br></div><br><div class="gmail_extra">-- <br><div class="m_-8990030224388909394gmail-m_-1622712601346819848gmail_signature">Alessandro Pasotti<br>w3:   <a href="http://www.itopen.it" target="_blank">www.itopen.it</a></div>
</div></div>