Skip to content

add byte counted text input#388

Merged
zjs81 merged 4 commits into
devfrom
msg-chars
Apr 20, 2026
Merged

add byte counted text input#388
zjs81 merged 4 commits into
devfrom
msg-chars

Conversation

@446564

@446564 446564 commented Apr 15, 2026

Copy link
Copy Markdown
Collaborator

adds a new byte counted text input widget

configurable limit and shows user both count and limit

provides color feedback

use new widget in chat and channel text entry

respect smaz encoding byte length in count when active

446564 and others added 2 commits April 15, 2026 09:04
adds a new widget that counts bytes during entry

configurable limit and shows user both count and limit

provides color feedback

use new widget in chat and channel text entry
@446564 446564 changed the title Msg chars add byte counted text input Apr 15, 2026
@446564 446564 requested a review from zjs81 April 15, 2026 16:13
@zjs81

zjs81 commented Apr 17, 2026

Copy link
Copy Markdown
Owner

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b572314ae9

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread lib/screens/chat_screen.dart Outdated
Comment thread lib/widgets/byte_count_input.dart
@zjs81

zjs81 commented Apr 17, 2026

Copy link
Copy Markdown
Owner

@446564 Could you please resolve what codex brought up?

@446564

446564 commented Apr 17, 2026

Copy link
Copy Markdown
Collaborator Author

@446564 Could you please resolve what codex brought up?

I'm not sure about the second claim, what do you think @zjs81? We don't touch any structured payload stuff and it doesn't use the textfield.

@zjs81 zjs81 merged commit 6b6a881 into dev Apr 20, 2026
6 checks passed
@446564 446564 deleted the msg-chars branch April 20, 2026 16:54
HDDen added a commit to HDDen/meshcore-open that referenced this pull request Apr 22, 2026
commit 609d0c8
Author: HDDen <62592944+HDDen@users.noreply.github.com>
Date:   Wed Apr 22 03:59:43 2026 +0300

    Added Cyr2Lat compression by replacing 2-byte cyrillic chars by 1-byte latin

commit 6b6a881
Merge: a4bbeff 8ef8a38
Author: zjs81 <30362347+zjs81@users.noreply.github.com>
Date:   Mon Apr 20 09:17:00 2026 -0700

    Merge pull request zjs81#388 from zjs81/msg-chars

    add byte counted text input

commit 8ef8a38
Author: ericz <erics.zimmermann@gmail.com>
Date:   Sat Apr 18 00:06:03 2026 +0200

    change to prepare Outbound Text Functions.

commit ddcda4b
Author: Enot (ded) Skelly <enot@mailbox.org>
Date:   Fri Apr 17 14:07:00 2026 -0700

    keep multiline editing

commit b572314
Author: ericz <erics.zimmermann@gmail.com>
Date:   Sat Apr 11 18:48:43 2026 +0200

    respect smaz encoding in message byte length calculation.

commit e97fb9b
Author: Enot (ded) Skelly <enot@mailbox.org>
Date:   Wed Apr 15 08:35:09 2026 -0700

    add byte counted text input

    adds a new widget that counts bytes during entry

    configurable limit and shows user both count and limit

    provides color feedback

    use new widget in chat and channel text entry
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