Skip to content
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

add description to explain Drive Prefix #377

Merged
merged 1 commit into from
Nov 14, 2023

Conversation

bobafetthotmail
Copy link
Contributor

@bobafetthotmail bobafetthotmail commented Nov 13, 2023

added a description so people know how to configure Drive Prefix setting for their printer.

I moved the Print/Delete buttons to the left because after I added the description row they were moved too much to the right. I could not figure out a better way to fix the alignment issue.

Screenshot_20231113_031502

qml/MonitorItem.qml Outdated Show resolved Hide resolved
qml/MonitorItem.qml Outdated Show resolved Hide resolved
added a description so people know how to configure the
Drive Prefix setting for their printer.

added missing anchors to fix a layout issue in the popup.
Since after adding the instruction label the print/delete buttons
were moved to the right and not visible anymore
Copy link

sonarcloud bot commented Nov 13, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@bobafetthotmail
Copy link
Contributor Author

Now it should be ok, I edited the label text and I used anchors.fill: parent to fix the anchor issue.

Since to fix this I had to open the file on my PC, the text editor program I use auto-deleted the "space bar characters" at the empty line 238 too.

@Jeredian Jeredian self-requested a review November 14, 2023 09:05
@Jeredian Jeredian merged commit 40d6d72 into PrintMakerLab:develop Nov 14, 2023
2 checks passed
@Jeredian
Copy link
Collaborator

closes #371

@Jeredian Jeredian mentioned this pull request Nov 28, 2023
@Jeredian Jeredian added the pull request: community Improvements or additions from the community label Nov 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pull request: community Improvements or additions from the community
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants