Skip to content

xev: add page #6548

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

Merged
merged 5 commits into from
Sep 18, 2021
Merged

xev: add page #6548

merged 5 commits into from
Sep 18, 2021

Conversation

Shinyzenith
Copy link
Contributor

  • [ ✅ ] The page (if new), does not already exist in the repository.
  • [ ✅ ] The page is in the correct platform directory (common/, linux/, etc.)
  • [ ✅ ] The page has 8 or fewer examples.
  • [ ✅ ] The PR title conforms to the recommended templates.
  • [ ✅ ] The page follows the content guidelines.
  • [ ✅ ] The page description includes a link to documentation or a homepage (if applicable).

@tldr-bot
Copy link

The build for this PR failed with the following error(s):

pages/linux/xev.md:6: TLDR005 Example descriptions should end in a colon with no trailing characters
pages/linux/xev.md:10: TLDR005 Example descriptions should end in a colon with no trailing characters

Please fix the error(s) and push again.

@pixelcmtd pixelcmtd changed the title xev:add page xev: add page Sep 18, 2021
Copy link
Member

@pixelcmtd pixelcmtd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this in the linux directory? Isn't it available on all systems with X?

@Shinyzenith
Copy link
Contributor Author

Why is this in the linux directory? Isn't it available on all systems with X?

Well android , osx and windows don't use Xorg.
And I have very limited knowledge in sunos so I thought it's best to put in linux and have a collaborator take a look at it. 😄

@pixelcmtd
Copy link
Member

Well android , osx and windows don't use Xorg.

xquartz.org

Screen.Recording.2021-09-18.at.13.51.05.mov

I also don't know a lot about Solaris, but I think, they at least support X.

Pages should only go in linux, if they are really Linux specific. But this applies to *BSD, Darwin and other Unixes.

@Shinyzenith
Copy link
Contributor Author

Didn't know xev worked on quartz display server. Point noted, moving file to common @pixelcmtd

@Shinyzenith
Copy link
Contributor Author

I hope that fixes all the issues with the page. 👍🏽

Copy link
Contributor

@marchersimon marchersimon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @AakashSharma7269

@Shinyzenith
Copy link
Contributor Author

Glad i can help <3

@pixelcmtd pixelcmtd merged commit 2dd1546 into tldr-pages:main Sep 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants