Skip to content

Conversation

@jrandolf-google
Copy link
Contributor

@jrandolf-google jrandolf-google commented Aug 22, 2023

Issue #494

File dialog PR: #568


Preview | Diff

@OrKoN
Copy link
Contributor

OrKoN commented Aug 25, 2023

@jgraham @whimboo @juliandescottes PTAL Perhaps you have suggestions for the open discussions and some other suggestions.

@jrandolf-google jrandolf-google force-pushed the jrandolf/set-files branch 4 times, most recently from 6b6ea1b to 22e6df6 Compare September 11, 2023 13:40
@jrandolf-google jrandolf-google force-pushed the jrandolf/set-files branch 4 times, most recently from 241213b to 4c827b8 Compare October 10, 2023 21:15
@jrandolf-google jrandolf-google requested a review from OrKoN October 10, 2023 21:15
@jrandolf-google jrandolf-google changed the base branch from main to jrandolf/file-dialog-opened October 11, 2023 08:31
@jrandolf-google jrandolf-google force-pushed the jrandolf/file-dialog-opened branch from 3bd59bb to e31b876 Compare November 8, 2023 11:34
@jrandolf-google jrandolf-google force-pushed the jrandolf/set-files branch 3 times, most recently from e24c9dc to f350765 Compare November 8, 2023 12:03
@OrKoN OrKoN force-pushed the jrandolf/file-dialog-opened branch 2 times, most recently from 5421b69 to 4e766f5 Compare November 21, 2023 14:00
@OrKoN OrKoN requested a review from sadym-chromium December 8, 2023 14:03
@OrKoN OrKoN force-pushed the jrandolf/set-files branch from 921398a to a1e64ec Compare December 8, 2023 14:04
@whimboo
Copy link
Contributor

whimboo commented Jan 5, 2024

Looks like this PR stalled a bit. @jrandolf mind giving a quick update what's the status?

@jrandolf-google
Copy link
Contributor Author

@whimboo Waiting on File dialog PR: #568

@jrandolf-google jrandolf-google changed the base branch from jrandolf/file-dialog-opened to main January 8, 2024 12:54
Co-authored-by: Alex Rudenko <[email protected]>
@jrandolf-google jrandolf-google force-pushed the jrandolf/set-files branch 3 times, most recently from d47c6c8 to 326b073 Compare January 10, 2024 18:27
Co-authored-by: Henrik Skupin <[email protected]>
Copy link
Contributor

@whimboo whimboo left a comment

Choose a reason for hiding this comment

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

Thanks for the updates. For me it looks fine. @jgraham can you please take a final look as well?

Co-authored-by: jgraham <[email protected]>
@whimboo
Copy link
Contributor

whimboo commented Jan 12, 2024

@jrandolf are you planning to write necessary wpt tests for this new feature?

@whimboo whimboo merged commit 804e6aa into main Jan 12, 2024
@whimboo whimboo deleted the jrandolf/set-files branch January 12, 2024 13:43
github-actions bot added a commit that referenced this pull request Jan 12, 2024
SHA: 804e6aa
Reason: push, by whimboo

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
sadym-chromium pushed a commit that referenced this pull request Feb 25, 2025
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.

5 participants