Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Right-Click in File Tree to Beautify other files #36

Closed
2 tasks done
Glavin001 opened this issue Jun 28, 2014 · 22 comments
Closed
2 tasks done

Right-Click in File Tree to Beautify other files #36

Glavin001 opened this issue Jun 28, 2014 · 22 comments

Comments

@Glavin001
Copy link
Owner

  • Right-click single file
  • Right-click directory
@Glavin001
Copy link
Owner Author

See atom/tree-view#44 for example.

@Glavin001 Glavin001 added this to the v1.0.0 milestone Jul 12, 2014
@Glavin001 Glavin001 self-assigned this Jul 12, 2014
@budhajeewa
Copy link

@Glavin001 "high priority" label? Awesome! 😄 👍

@Glavin001
Copy link
Owner Author

I have midterms these next two weeks, not to mention work, however I hope to have a few minutes of free time where I could get this completed for you 😃.

@budhajeewa
Copy link

You're great. However, I believe we can wait two weeks. Do the best in exams Glavin. :)

@Glavin001 Glavin001 modified the milestones: My Christmas List, v1.0.0 Dec 27, 2014
@Glavin001
Copy link
Owner Author

Wow time flies! Sorry about the delay: work has been as crazier than exams 😜. Hoping to get this done for the holidays.

@Glavin001
Copy link
Owner Author

So single-file right-click in Atom's tree to beautify is working, and I am using it often actually! (not yet released, coming soon) Now I thinking about the directory beautification support.

I am thinking of using https://www.npmjs.com/package/node-dir and recursively processing the files in sub-directories, and if they are a supported Grammar then beautify them, and if not, ignore the file. Thoughts? How else could this be done, from a users perspective. Possibly list all of the files in the directory and sub-directories and allow the user to select/check files using multi-select-box.

@Glavin001
Copy link
Owner Author

Single-File support has been published to v0.18.0. Happy New Year! 😃

@budhajeewa
Copy link

Wow great @Glavin001 ! I am eagerly waiting for the directory support. It should be recursive. :)

@Glavin001
Copy link
Owner Author

Published to v0.19.0! Enjoy :)

@budhajeewa
Copy link

@Glavin001 You are awesome! 👍

@budhajeewa
Copy link

Hi @Glavin001 , I checked this, but I encounter an error when trying to get it to work. Please see #170.

@luisrudge
Copy link
Contributor

This is not showing as option anymore :(

@Glavin001
Copy link
Owner Author

I have temporarily removed this option. See #213

@IanShoe
Copy link

IanShoe commented Oct 1, 2015

Just curious if any progress has been made since the concerns from #213?

@Glavin001
Copy link
Owner Author

@IanShoe : Not yet completed. Would definitely review a Pull Request implementing the feature list here: #213 (comment)

@brunolemos
Copy link

@Glavin001 please add an option to easily do that. You could make an alert with the dangerous risk or open all files without saving it so the user can check it.

@bs-thomas
Copy link

@Glavin001 Sorry not trying to rush you. I was really looking forward to this feature actually. May I ask if there are any updates on this? Thank you very much for your time and kind attention!

@Glavin001
Copy link
Owner Author

Hi everyone! Sorry, I haven't had time to implement this. The option could be easily added back ( see f4b2916 ), however I would want there to be safety measures to ensure users do not lose their code, in the case that Atom and/or Atom-Beautify locks up and starts deleting data.
I would be happy to review a Pull Request and have further discussions about this feature. Thanks!

@julkue
Copy link

julkue commented Sep 11, 2016

What is the preferred workaround in the meantime (for years)? Manually beautifying isn't really an option with many files.

@Glavin001
Copy link
Owner Author

@julmot : you could edit the menus/atom-beautify.cson file and uncomment the Beautify Directory command: f4b2916
I believe Atom packages are installed in ~/.atom/packages directory. So look for for a file with a possible path of ~/.atom/packages/atom-beautify/menus/atom-beautify.cson and after you edit the file reload Atom.

@Glavin001
Copy link
Owner Author

Actually the preferred workaround is not a workaround and instead that someone submits a Pull Request to Atom-Beautify resolving the problems outlined in #213 😄 . I need someone who wants this feature to exist and is willing to contribute to ensure it is developed and supported. I need your help and contribution.

I am currently focusing my after-work-and-after-school hours on #1174 which will provide a lot of value to both users and maintainers. Beautifying a directory of files is on the todo list to be completed after #1174 has been completed. However, no ETA on this feature. The best way to get this feature implemented is to contribute! Thanks in advance!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants