Update docs/qt/architecture.md to fix typos and clarify language (PR #26261)
Hamish Mcintyre-Bhatty
Just a quick PR to fix some typos and clarify some of the language here, as the English was unclear in a few places.
You can view, comment on, or merge this pull request online at:
https://github.com/wxWidgets/wxWidgets/pull/26261
Commit Summary
- 524c65b Update docs/qt/architecture.md to fix typos and clarify language
File Changes
(1 file)
- M docs/qt/architecture.md (14)
Patch Links:
- https://github.com/wxWidgets/wxWidgets/pull/26261.patch
- https://github.com/wxWidgets/wxWidgets/pull/26261.diff
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
AliKet
AliKet left a comment (wxWidgets/wxWidgets#26261)Thanks, but some of the information here are outdated and no longer valid. I'll take a look soon...
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Hamish Mcintyre-Bhatty
hamishmb left a comment (wxWidgets/wxWidgets#26261)Ah okay. Should I still leave this open?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
AliKet
AliKet left a comment (wxWidgets/wxWidgets#26261)Ah okay. Should I still leave this open?
I'll create a new PR that includes both your changes and mine, so feel free to close this one when the new one is ready.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Hamish Mcintyre-Bhatty
hamishmb left a comment (wxWidgets/wxWidgets#26261)Okay, just let me know.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
VZ
vadz left a comment (wxWidgets/wxWidgets#26261)@AliKet I've reassigned this to you, not sure if this allows you to close it yourself if this is the right thing to do. If not, please let me know what should be done. TIA!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
VZ
vadz left a comment (wxWidgets/wxWidgets#26261)One month later, I think I'm going to apply this, finally, as it should do no harm to fix the wording problems in this file.
@AliKet Please let me know if you don't want me to do it, i.e. if you already have some changes conflicting with the ones here.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.
AliKet
AliKet left a comment (wxWidgets/wxWidgets#26261)One month later, I think I'm going to apply this, finally, as it should do no harm to fix the wording problems in this file.
@AliKet Please let me know if you don't want me to do it, i.e. if you already have some changes conflicting with the ones here.
Sorry for the delay due to lack of time.
Yes, go ahead and apply this — I'll rebase my changes on it later anyway.
TIA!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
VZ
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Hamish Mcintyre-Bhatty
hamishmb left a comment (wxWidgets/wxWidgets#26261)Cheers!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.