examples: add basic widgets example #622
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ahayzen-kdab
force-pushed
the
577-extern-cpp-qt-block
branch
2 times, most recently
from
d33623c
to
27851e2
Compare
ahayzen-kdab
commented
Jul 31, 2023
ahayzen-kdab
force-pushed
the
577-extern-cpp-qt-block
branch
from
27851e2
to
81a04e6
Compare
ahayzen-kdab
force-pushed
the
577-extern-cpp-qt-block
branch
from
81a04e6
to
461bfaf
Compare
ahayzen-kdab
changed the title
ahayzen-kdab
commented
Jan 15, 2024
ahayzen-kdab
commented
Jan 15, 2024
| type QString = cxx_qt_lib::QString; | ||
|
|
||
| #[qsignal] | ||
| #[allow(dead_code)] |
There was a problem hiding this comment.
as we only connect to the signal and don't emit it causes dead code, wonder if we should always mark as dead ?
There was a problem hiding this comment.
(as this is an existing signal rather than one being created from Rust, so for inherit / C++Qt types you might not see an emit)
ahayzen-kdab
force-pushed
the
577-extern-cpp-qt-block
branch
from
461bfaf
to
8574b42
Compare
ahayzen-kdab
force-pushed
the
577-extern-cpp-qt-block
branch
from
8574b42
to
525e1df
Compare
Merged
ahayzen-kdab
force-pushed
the
577-extern-cpp-qt-block
branch
from
525e1df
to
58abe41
Compare
ahayzen-kdab
commented
Apr 16, 2024
| use cxx_qt_lib::QString; | ||
| use cxx_qt_lib_extras::{QApplication, QPushButton}; | ||
|
|
||
| fn main() { |
There was a problem hiding this comment.
we should have some kind of test as well, even if C++ to see if this works, unsure how so far....
ahayzen-kdab
force-pushed
the
577-extern-cpp-qt-block
branch
3 times, most recently
from
bcb4e07
to
b9dcbfc
Compare
ahayzen-kdab
force-pushed
the
577-extern-cpp-qt-block
branch
from
b9dcbfc
to
dae15d9
Compare
ahayzen-kdab
commented
Apr 17, 2024
| #[qobject] | ||
| type QPushButton; | ||
|
|
||
| // TODO: we should use upcasting methods here and implement QAbstractButton and QWidget |
There was a problem hiding this comment.
we should figure out how upcasting is going to work, as otherwise we are going to duplicate a load of methods in the widgets world
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Lots of hacks, but we can launch a
QPushButton!I'll likely pick things off from this branch as separate pull requests over time.