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

This PR is fixing the bugs introduced by merging PR #1487 #1526

Merged
merged 29 commits into from
Jan 21, 2021

Conversation

kisslorand
Copy link
Contributor

@kisslorand kisslorand commented Jan 17, 2021

Requirements

MKS or BTT TFT

Description

This is a bugfix for the bugs introduced by PR #1487.
The fixed bugs are:

  • filament length not calculated when printing from remote host and from onboard SD
    NoLength

  • in some cases, prinintg from remote host and onboard SD, the file name is missing from the print summary
    NoName

  • other languages than English are not taken into account properly after print finished
    PrintFinishLanguage

  • in some cases, printing from remote host from onboard SD, after print finished cannot exit from the menu, screen freezes

Freeze.mp4

Benefits

Everyone

Additional info

This PR does not include any extra popup, there's not user configurable parameter added.
Extra RAM usage: 0 (zero)
Extra flash usage: ~72 bytes

kisslorand and others added 29 commits November 28, 2020 12:39
@kisslorand
Copy link
Contributor Author

kisslorand commented Jan 17, 2021

@AntoszHUN
Please check the little change I made in the Hungarian language file. My intention was to make the file opening confirmation popup look better in case of very long filenames.
Thank you in advance.

@AntoszHUN
Copy link
Contributor

@AntoszHUN
Please check the little change I made in the Hungarian language file. My intention was to make the file opening confirmation popup look better in case of very long filenames.
Thank you in advance.

You shouldn't always vary with it. Then there would be no problem. Why wasn't it as good as it was? Now what should I look at?

@AntoszHUN
Copy link
Contributor

I watched. Will be good. I'll test it and see if it gets into the master.

@kisslorand
Copy link
Contributor Author

Thank you.

@kisslorand
Copy link
Contributor Author

@bigtreetech
Please consider merging this so other PRs will have a base without the bugs this PR fixes.

@bigtreetech bigtreetech merged commit fb5823b into bigtreetech:master Jan 21, 2021
Uz45 added a commit to Uz45/BIGTREETECH-TouchScreenFirmware that referenced this pull request Jan 25, 2021
adds keyboard layout for quertz awerty bigtreetech#1550
Correction CZ translation bigtreetech#1546
FIX bigtreetech#1539 bigtreetech#1541
Fix get filesize and cancle print for RepRapFirmware bigtreetech#1533
This PR is fixing the bugs introduced by merging PR bigtreetech#1487 bigtreetech#1526
update german translation and add missing icons bigtreetech#1525
Add missing icons and Hungarian language bigtreetech#1520
Added Home Offset menu + bugfixes and cleanup bigtreetech#1518
Uz45 added a commit to Uz45/BIGTREETECH-TouchScreenFirmware that referenced this pull request Jan 31, 2021
adds keyboard layout for quertz awerty bigtreetech#1550
Correction CZ translation bigtreetech#1546
FIX bigtreetech#1539 bigtreetech#1541
Fix get filesize and cancle print for RepRapFirmware bigtreetech#1533
This PR is fixing the bugs introduced by merging PR bigtreetech#1487 bigtreetech#1526
update german translation and add missing icons bigtreetech#1525
Add missing icons and Hungarian language bigtreetech#1520
Added Home Offset menu + bugfixes and cleanup bigtreetech#1518
Readme.md update and format fix bigtreetech#1557
Reduce global variables and cleanup bigtreetech#1558
Added the possibility to change to time the bootscreen logo is shown bigtreetech#1560
Beep when temp reached bigtreetech#1561
Added new param in FeatureSettings menu + toggle ACK in Terminal menu… bigtreetech#1563
jeffeb3 pushed a commit to V1EngineeringInc/BIGTREETECH-TouchScreenFirmware that referenced this pull request Nov 10, 2021
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.

3 participants