Thanks to visit codestin.com
Credit goes to github.com

feat: Implement ext_* Protocol and DBus Security #206

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.

Sign up for GitHub

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

Jump to bottom

Merged

Decodetalkers merged 18 commits into waycrate:main from Rishik-Y:Submission

Nov 12, 2025

Contributor

Rishik-Y commented Aug 16, 2025

No description provided.

Rishik-Y added 11 commits

June 12, 2025 19:42


          Modified wayshot

018e5de


          Merge remote-tracking branch 'origin/GSoC' into GSoC

d33d2de

# Conflicts:
#	Cargo.lock


          Added get color

a81d9d2


          Made wlr to xdg support

02c25c2


          Rustfmt and some fixes

bc1d257

Added Scale and xdg_output to OutputInfo


           Quality changes to various parts of the code

48602e0


          Added Proper Notifications

d24ff8f


          Testing Streaming Feature

58fdcb7


          Removing Changes for submission

1291c17


          Returning back to wlr from xdg

8743ece


          Added stdout and removed streaming from wayshot.rs

b11d09a

Clippy Fix && Rustfmt

RustFMT

Rishik-Y marked this pull request as draft

August 16, 2025 13:51


          Merge branch 'main' into Submission

19aa1f7

Rishik-Y marked this pull request as ready for review

August 16, 2025 21:15

Decodetalkers reviewed

View reviewed changes

libwayshot/src/lib.rs Outdated Show resolved Hide resolved

Decodetalkers reviewed

View reviewed changes

libwayshot/src/lib.rs Outdated Show resolved Hide resolved

Decodetalkers reviewed

View reviewed changes

libwayshot/src/lib.rs Outdated Show resolved Hide resolved

Decodetalkers reviewed

View reviewed changes

libwayshot/src/lib.rs Outdated Show resolved Hide resolved

Decodetalkers reviewed

View reviewed changes

libwayshot/src/ext_image_protocols.rs Outdated Show resolved Hide resolved

Decodetalkers requested review from Shinyzenith and uncomfyhalomacro

August 17, 2025 03:00

Decodetalkers reviewed

View reviewed changes

wayshot/src/ext_wayshot.rs Outdated Show resolved Hide resolved

Decodetalkers requested changes

View reviewed changes

wayshot/src/ext_wayshot.rs Outdated Show resolved Hide resolved

Decodetalkers reviewed

View reviewed changes

libwayshot/src/ext_image_protocols.rs Outdated Show resolved Hide resolved

Decodetalkers reviewed

View reviewed changes

libwayshot/src/dispatch.rs Outdated Show resolved Hide resolved

Decodetalkers reviewed

View reviewed changes

libwayshot/src/lib.rs Outdated Show resolved Hide resolved

libwayshot/src/lib.rs Outdated Show resolved Hide resolved

Member

Shinyzenith commented Aug 17, 2025

@Rishik-Y mark the PR as draft for now till you get the code organized and cleaned up.

Rishik-Y marked this pull request as draft

August 17, 2025 05:07

Rishik-Y force-pushed the Submission branch from 6d13858 to a24452c Compare

August 17, 2025 05:46


          RustFMT

0f78f1a

Renamed WayshotError back to Error

Removed println!

Renamed output back to wl_output

Image removed

RustFMT

Rarranged properly

RustFmt

Typo's fix

Rishik-Y added 2 commits

September 4, 2025 02:08


          Merge branch 'main' into Submission


          Rustfmt

db2ceac

Rishik-Y force-pushed the Submission branch from 9a1d21f to db2ceac Compare

September 3, 2025 20:50

Rishik-Y marked this pull request as ready for review

September 3, 2025 21:00

Collaborator

Decodetalkers commented Oct 16, 2025

Ok, let me try this pr

Collaborator

Decodetalkers commented Oct 16, 2025

but I think the notify part should be on the other pr..

Collaborator

Decodetalkers commented Oct 16, 2025

The screenshot not work

Collaborator

Decodetalkers commented Oct 16, 2025

Do not split the ext part and the origin part, let the libwayshot to decide which one to use. make them the same

Decodetalkers reviewed

View reviewed changes

libwayshot/src/ext_image_protocols.rs Outdated Show resolved Hide resolved

Decodetalkers reviewed

View reviewed changes

wayshot/src/wayshot.rs Outdated Show resolved Hide resolved

Collaborator

Decodetalkers commented Oct 16, 2025

I do not think this pr is usable... now..

Collaborator

Decodetalkers commented Oct 16, 2025

sorry, I want let me to finish this pr, thanks for your efforts...

Rishik-Y marked this pull request as draft

November 1, 2025 08:14

Collaborator

Decodetalkers commented Nov 3, 2025

ok, as @Shinyzenith said, you can take my pr as example, and continue to finish it. It will be ok for you to copy that pr. And I still do not finish the toplevel part. You can think about how to design it

Rishik-Y added 2 commits

November 5, 2025 15:29


          Made Additional Changes according Decode's PR Code

c45fb0f


          Made Better handling for Toplevel listing

52e22d0

Rishik-Y force-pushed the Submission branch from cf4342a to 52e22d0 Compare

November 7, 2025 16:18

Rishik-Y marked this pull request as ready for review

November 7, 2025 16:24

Decodetalkers reviewed

View reviewed changes

wayshot/Cargo.toml Show resolved Hide resolved

Decodetalkers reviewed

View reviewed changes

libwayshot/src/lib.rs Outdated Show resolved Hide resolved

Rishik-Y force-pushed the Submission branch 2 times, most recently from 3c55917 to 8369dc7 Compare

November 8, 2025 19:36


          Modifed Notification structure to support config

b2330c6

Rishik-Y force-pushed the Submission branch from 433b641 to b2330c6 Compare

November 8, 2025 19:45

Rishik-Y requested a review from Decodetalkers

November 8, 2025 23:05

Collaborator

Decodetalkers commented Nov 9, 2025

I mean we do not need notification... It is not in plan

Collaborator

Decodetalkers commented Nov 9, 2025

ok, I think it is good

Decodetalkers approved these changes

View reviewed changes

Collaborator

Decodetalkers commented Nov 9, 2025

@Shinyzenith can we merge it?

Decodetalkers merged commit df13eb4 into waycrate:main

4 checks passed

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

Labels

None yet