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.

Sign up for GitHub

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

Jump to bottom

Printer information menu #4188

Merged
merged 6 commits into from
Jul 2, 2016
Merged

Printer information menu #4188

merged 6 commits into from
Jul 2, 2016

Conversation

thinkyhead
Copy link
Member

Adds an "About Printer" menu entry in the main menu, to show basic information about the printer and firmware configuration.

Mildly tweaked #4175, ready to merge.

birkett and others added 6 commits July 2, 2016 15:22
@birkett @thinkyhead
Add "About Printer" information menu.
7a377c1 
Includes:
  *firmware version / branch / date.
  *extruder count
  *board information (name, serial details, power supply type)
  *thermistors (names, min/max temperatures)
  *printer statistics (PRINTCOUNTER details)

Thanks to @thinkyhead for contributions.
@thinkyhead
Squish sanity check for build info
dbfc87c
@thinkyhead
Tweak spacing in printcounter.cpp
fa9b2a9
@thinkyhead
Static menu items take an optional string, LJ by default
439e781
@thinkyhead
Patch up About Printer sub-menus
51b7820
@thinkyhead
Add a Travis test for LCD_INFO_MENU
b77069d
@thinkyhead thinkyhead added this to the 1.1.0 milestone Jul 2, 2016
@thinkyhead thinkyhead merged commit 4a8e191 into MarlinFirmware:RCBugFix Jul 2, 2016
@thinkyhead thinkyhead deleted the rc_birkett_info_menu_jul2 branch July 2, 2016 22:52
@birkett birkett mentioned this pull request Jul 3, 2016
Closed
@ghost
Copy link

ghost commented Jul 3, 2016

I found a problem of displaying.
In "About Printer" section, minutes of "Total Print Time"(MSG_INFO_PRINT_TIME) is displayed on wrong place.
dscn1637

But when I shorten the MSG_INFO_PRINT_TIME, minutes is displayed on right place.
dscn1638

@thinkyhead
Copy link
Member Author

thinkyhead commented Jul 3, 2016 •
edited
Loading

This is going to be a general problem with this feature. The labels are too long. They should be more readable too…

Print Count : 7
Finished    : 10
Total Time  : 01:00

And for 16-width displays…

Prints   : 7
Finished : 10
Duration : 01:00

@thinkyhead thinkyhead mentioned this pull request Jul 3, 2016
Merged
@petrzjunior petrzjunior mentioned this pull request Jul 6, 2016
Merged
@thinkyhead thinkyhead mentioned this pull request Jul 8, 2016
Closed
drewmoseley pushed a commit to drewmoseley/Marlin that referenced this pull request Nov 8, 2023
@3d-gussner
Merge pull request MarlinFirmware#4188 from leptun/fix_missing_live_a…
719d170 
…djust_z_settings

Always allow Live Adjust Z in Settings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
PR: New Feature
Projects
None yet
Milestone
1.1.0
Development

Successfully merging this pull request may close these issues.
2 participants
@thinkyhead @birkett