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 printers page UI mock-up #134

Draft
wants to merge 28 commits into
base: master
Choose a base branch
from
Draft

Add printers page UI mock-up #134

wants to merge 28 commits into from

Conversation

Feichtmeier
Copy link
Member

@Feichtmeier Feichtmeier commented Oct 5, 2021

Currently only UI with an empty model and not connected
image
image

Ref #55

@hellsworth
Copy link

I think it looks great!! My only criticism is the wording "Open Queue" for the button.. If there are no active jobs, then I prefer how g-c-c currently does it.. the button is grey'd out and it says "No active jobs" (see below)

Screenshot from 2021-10-15 10-06-54

@Feichtmeier
Copy link
Member Author

@hellsworth @madsrh

image
image

Please let the feedback roll/flow :)

@hellsworth
Copy link

I like the updated look @Feichtmeier :)

@Feichtmeier Feichtmeier changed the title WIP Add printers page Add printers page UI mock-up Feb 19, 2022
@Feichtmeier Feichtmeier marked this pull request as ready for review February 19, 2022 20:10
@Feichtmeier Feichtmeier marked this pull request as draft February 19, 2022 20:44
@Feichtmeier
Copy link
Member Author

grafik

😆

@madsrh
Copy link
Collaborator

madsrh commented Aug 6, 2023

Great printer names 😅
I preferred the icon on the left and settings on the right, but maybe there's a reason you've moved them.

@Feichtmeier
Copy link
Member Author

Great printer names 😅
I preferred the icon on the left and settings on the right, but maybe there's a reason you've moved them.

Oh actually not. Was trying to get it back to work. Thanks will change it back

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.

4 participants