-
Notifications
You must be signed in to change notification settings - Fork 1
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
#54-Subheader component #61
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you go through and replace all our subheaders with this in the pit screen and send the screen shot please
NERODevelopment/CMakeLists.txt
Outdated
@@ -48,7 +48,9 @@ set(PROJECT_SOURCES | |||
|
|||
find_package(Qt6 REQUIRED COMPONENTS Core Gui Qml Quick Network Mqtt) | |||
|
|||
qt_add_executable(NEROApp ${PROJECT_SOURCES}) | |||
qt_add_executable(NEROApp ${PROJECT_SOURCES} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
revert this change
NERODesign/NERO.qmlproject.qtds
Outdated
@@ -61,8 +61,8 @@ | |||
<variable>ProjectExplorer.Project.Target.0</variable> | |||
<valuemap type="QVariantMap"> | |||
<value type="QString" key="DeviceType">Desktop</value> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dont include this file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also remember to link PR and assign yourself
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
undo changes to qmlproject.qtds
@@ -0,0 +1,26 @@ | |||
import QtQuick 2.15 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
delete entire file
NERODevelopment/CMakeLists.txt
Outdated
@@ -85,5 +85,3 @@ install(TARGETS NEROApp | |||
LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR} | |||
RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR} | |||
) | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
undo changes
This reverts commit a554623.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
amazing
Changes
Created the Subheader component
Notes
Used qml properties to take in the font type to be loaded from a file and text
Test Cases
N/A
Screenshots
To Do
n/a
Checklist
It can be helpful to check the
Checks
andFiles changed
tabs.Please review the contributor guide and reach out to your Tech Lead if anything is unclear.
Please request reviewers and ping on slack only after you've gone through this whole checklist.
Closes issue #54