search

jccguimaraes / atom-project-viewer

Project Viewer - An Atom's package
MIT License
75 stars 15 forks source link

tidy editor adn address issue #197 #180

Closed bobtherobot closed 6 years ago

bobtherobot commented 7 years ago

Pull Request

Tidy up and simplify editor panel. Addressed issue 197 confusion.

Reason

Tidy up and simplify editor panel. Addressed issue 197 confusion. ss screen shot 2017-11-05 at 5 16 30 am screen shot 2017-11-05 at 5 18 24 am screen shot 2017-11-05 at 5 17 59 am

and if possible provide a screenshot. :+1:

jccguimaraes commented 7 years ago

amazing work @bobtherobot, let me see if later in the day I can have a real good look at it 👍

jccguimaraes commented 7 years ago

I actually don't like the non-styled headers as well as the buttons not being fixed at top :( But the rest seems cozy

editor_styles

jccguimaraes commented 7 years ago

There is some inconsistency between opening a new editor and creating a new one from the workspace vs side panel. Also, i don't see any option like copy project as discussed in #179! Am i missing something?

Current

editor_1

PR

editor_2

jccguimaraes commented 7 years ago

One last thing I want to point out. As described in #176, editor will stay as is in current version, so we need to make sure this suites all needs 👍 ?