Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Should Screenshotting be removed from Neofetch? #1001

Closed
dylanaraps opened this issue May 21, 2018 · 2 comments
Closed

Should Screenshotting be removed from Neofetch? #1001

dylanaraps opened this issue May 21, 2018 · 2 comments

Comments

@dylanaraps
Copy link
Owner

dylanaraps commented May 21, 2018

I’m going through a purge of the Neofetch source. Removing anything that
does not belong. I stumbled across the screenshot functions and thought to
myself; Why are they even there in the first place?

Taking a screenshot is as easy as neofetch; scrot (or equivalent
tool
). Is -s/--scrot really necessary? We’re duplicating
functionality here.

This also extends to the feature of uploading screenshots to teknik.io
and imgur. This feature is not as easy to replicate
outside of Neofetch but it definitely does not belong.

What are your thoughts on this matter? Should the functionality stay or
go? 👍 or 👎 this post and comment below if you’d like to add your
view to the discussion.

  • Remove the feature: 👍
  • Keep the feature 👎
@iandrewt
Copy link
Contributor

I’ve never used it myself nor seen it used, and Neofetch is a system Information tool, not a screenshotting tool. It can go imo

@dylanaraps
Copy link
Owner Author

I've removed the feature from master but I'll keep this open to keep the discussion going.

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

No branches or pull requests

2 participants