[eclipse-arduino-dev] Re: I believe I have it!

  • From: Roberto Lo Giacco <rlogiacco@xxxxxxxxx>
  • To: eclipse-arduino-dev@xxxxxxxxxxxxx
  • Date: Thu, 24 Mar 2016 01:08:36 +0100

On Wed, Mar 23, 2016 at 7:11 PM, Jan Baeyens <jan@xxxxxxxxxx> wrote:

Still I believe a uniform path resolution helper can be useful to help
avoid troubles in the future.

I'm not sure what you mean by this.
Now ConfigurationPreferences contains some static methods
getInstallationPath();
getInstallationPathLibraries();
......
I'm open for suggestions to handle this differently


​That's exactly was I was suggesting, my bad I didn't see it before! +1​






Op 23/03/2016 om 2:13 schreef Roberto Lo Giacco:

You are right, I hadn't pull the last changes from the parent repository.
I haven't done an extensive verification, but it seems the problem I was
having is already fixed. Still I believe a uniform path resolution helper
can be useful to help avoid troubles in the future.

Please have a look at the following animated gifs, representing two
possible ways to browse into the library manager as a tree and selecting
the version to install: one makes use of right click buttons and its
already working as intended (even if on fake data, for simplicity), the
other one uses drop downs, but it's still not finished.

I believe we wish to have the one using drop downs, but if you prefer the
pup up menu just let me know.

In both cases I intend to add a search bar capable to perform full text
search on the tree like the "type filter text" available on the right pane
(should be simple).

PopUp Menu (right click)

[image: Inline image 1]


DropDown (still requiring some work)
[image: Inline image 2]

On Tue, Mar 22, 2016 at 10:56 PM, Jan Baeyens <jan@xxxxxxxxxx> wrote:

I can be wrong but ....  I thought I already fixed that one.


Op 22/03/2016 om 22:52 schreef Roberto Lo Giacco:

While trying to convert the library manager preference from a table to a
tree (not as easy as i thought) I believe I have found a workaround for the
issue with spaces in the installation path.

I'm creating an helper class InstallPathHelper which should provide a
single entry point to resolve the installation path and anything below that.

The main trick is not to use either URL or URI representation of the
path: those would require a lot of escaping and fixing. Instead
using Platform.getInstallLocation().toString() seems working nicely on my
Win10.

I'm currently working only within the Manager and related classes, but I
believe it can be used in multiple places and having one single entry point
to it should simplify fixing future path issues.

Would you agree?





GIF image

GIF image

Other related posts: