feat: hide message tracing#216
Conversation
|
I would like to see the setting moved into the chat settings dialog. along with adding a button and dialog for the channels. |
|
@wel97459 How do I find the chat settings dialog? Not sure where that one is. Where would the button go for channels? |
| borderRadius: BorderRadius.circular(12), | ||
| child: GifMessage( | ||
| url: | ||
| 'https://media.giphy.com/media/$gifId/giphy.gif', |
There was a problem hiding this comment.
This code shouldnt be in here. We should be using the gif widget helpers.
There was a problem hiding this comment.
I understand that it was before but we should move it to clean it up.
There was a problem hiding this comment.
I usually try to keep refactoring PRs separate but I'll do it here if you have a strong opinion. I think the chat/channel_chat could benefit from a more thorough widget refactoring later, we could hit it there too.
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 35498c1b90
ℹ️ 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".
|
Thanks for all the feedback. I’ll circle back later tonight or tomorrow. On the road to Elko NV to help them get their mesh set up. The gray cloud - was looking for a way to indicate hears by 0 repeaters (gray cloud/busy) vs >=1 (green check). Just something to indicate that it hit at least 1 repeater. I’m not quite sure when to consider a message sent except maybe >=1 ack. |
Oh sweet good luck out there. That makes sense I think maybe an empty check box would make more sense or something we can think about it. |
|
I assume the checks show that it has seen at least 1 relay. I can hardly see the difference in color and for people with color blindness this is even more of an issue. Ontop of the color could we have 1 check mark for sent, and 2 for when we have heard an echo? |
This PR introduces a more compact version of messages.
Closes #97
2026-02-22_11-25-08.mp4