Skip to content

Conversation

@Quency-D
Copy link
Contributor

@Quency-D Quency-D commented Sep 2, 2025

I have tested that my proposed changes behave as described.
I have tested that my proposed changes do not cause any obvious regressions on the following devices:

  • heltec-meshpocket
  • heltec-t114

const char* getSettingName(int i) const override;
const char* getSettingValue(int i) const override;
bool setSettingValue(const char* name, const char* value) override;
};
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are there plans to customise SensorManager? If not, you can just declare:
extern SensorManager sensors;

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The sensor section has been removed.

display.print(str);
}

void GxEPDDisplay::fillRect(int x, int y, int w, int h) {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just wondering why offset_x and offset_y are only applied in setCursor()?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

offset_x and offset_y only affect the text cursor position, making it easy to uniformly adjust the text area, while graphics drawing maintains the original coordinates for precise control.

@Quency-D
Copy link
Contributor Author

Quency-D commented Sep 4, 2025

It has been re-adapted according to commit #722.

@fdlamotte
Copy link
Collaborator

Still works good on TEcho, no regressions from the changes ... good work !

@Quency-D you might consider adding the following flags for meshpocket

  • -D AUTO_OFF_MILLIS=0 to get an always-on display (refresh is done only when needed now)
  • -D DISABLE_DIAGNOSTIC_OUTPUT this one will make GxEPD less verbose ;)

@Quency-D
Copy link
Contributor Author

Quency-D commented Sep 4, 2025

Still works good on TEcho, no regressions from the changes ... good work !

@Quency-D you might consider adding the following flags for meshpocket

  • -D AUTO_OFF_MILLIS=0 to get an always-on display (refresh is done only when needed now)
  • -D DISABLE_DIAGNOSTIC_OUTPUT this one will make GxEPD less verbose ;)

Thank you very much for your suggestions, that's great. I've modified both.

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