forrest-app / forrest

An npm scripts desktop client
MIT License
341 stars 33 forks source link

Feature: Collapsable script dividers in project view #79

Open mischah opened 8 years ago

mischah commented 8 years ago

With script dividers I mean those rows labelled with »Default commands« and »Customs scripts«.

It would be cool, if I could collapse the default commands. So I wouldn’t need to scroll all the way down to reach my customs scripts 😘

stefanjudis commented 8 years ago

Ja, this was the initial implementation. But I removed at some point, I think we would have to think of a way to make this configurable.

What about a settings section where you can set the order of the two sections and maybe some checkmarks to decide what "common" commands should be included?

mischah commented 8 years ago

Mhm. It kinda make sense to have the default commands on top.

What about a settings section where you can set the order of the two sections and maybe some checkmarks to decide what "common" commands should be included?

That’s a good idea 👌

But don’t you sing the collapsing thing is a good idea? Why not?

stefanjudis commented 8 years ago

Ja I removed because I was not sure how to handle it completely. Should it be always collapsed. Should the collapsed state be stored?

For me it's not a big deal as I control it via keyboard only (arrow keys). :)

mischah commented 8 years ago

Ja I removed because I was not sure how to handle it completely. Should it be always collapsed. > Should the collapsed state be stored?

Got the point. In my point of few the state should be stored per project. Plus nothing should be collapsed by default (after adding a new project).

For me it's not a big deal as I control it via keyboard only (arrow keys). :)

Ah. Didn’t checked keyboard accessibility yet. Gonna give it a try.

DanielRuf commented 5 years ago

Hi, is this feature request still valid or should we keep using the current solution?