i3 / i3.github.io

The i3wm.org website
https://i3wm.org/
191 stars 60 forks source link

Added link to i3status. Removed duplicate link to Debugging. #97

Closed snprajwal closed 3 years ago

snprajwal commented 3 years ago

I have done the edits in docs/4.9 and the /docs folder since I was unsure which one is currently in use on the website. This is my first ever PR, so please excuse if I've done something incorrectly in the process.

orestisfl commented 3 years ago

I am not really sure about this change

  1. i3status is not part of i3
  2. debugging is useful to be visible for both developers and users since it is meant to be done by both
Airblader commented 3 years ago

@snprajwal First of all thanks for the suggestion and the PR!

I have done the edits in docs/4.9 and the /docs folder since I was unsure which one is currently in use on the website.

Can you undo the changes to docs/4.9? Changing it in the current one should be good enough.

@orestisfl

i3status is not part of i3

That's true, but they are closely related. I know I've had trouble finding the i3status page before as well. Of course we could completely reorganize the website, but I think we get a reasonable benefit for close to no effort here, so I'd go with it.

debugging is useful to be visible for both developers and users since it is meant to be done by both

I agree, and I'd keep both for now as well.

snprajwal commented 3 years ago

@Airblader I have undone the changes to docs/4.9

  • debugging is useful to be visible for both developers and users since it is meant to be done by both

@orestisfl I simply thought that since both the "For users" and "For developers" links are displayed on the same page, it would be kinda redundant to have the exact same link twice. I am fairly confident devs will at least look at the links at the User section, which is why I left that link there :) @Airblader Let me know if I have to undo the removal

stapelberg commented 3 years ago

Let’s leave the debugging link in both sections. It’s one of the most central documents, and when people are upset because things don’t work, we want them to find it no matter in which section they happen to be looking.

stapelberg commented 3 years ago

Thanks. Can you also properly revert your change to docs/4.9/index.html please? You’re still changing whitespace in this pull request.

snprajwal commented 3 years ago

@stapelberg it's done. Sorry, looks like my autoindenting did that.

stapelberg commented 3 years ago

Thanks!