Skip to content

Updated :: Logging Module - UI Font & Sharable Logs#1

Merged
FeernandoOFF merged 1 commit intoTapadoo:mainfrom
FeernandoOFF:main
Nov 10, 2025
Merged

Updated :: Logging Module - UI Font & Sharable Logs#1
FeernandoOFF merged 1 commit intoTapadoo:mainfrom
FeernandoOFF:main

Conversation

@FeernandoOFF
Copy link
Collaborator

Improved UI for Logging Module

  • Smaller font size
  • Mono Spaced for better readability
  • Toggle large logs
  • Long-press to share logs
Screen.Recording.2025-11-07.at.12.49.39.mov

@FeernandoOFF FeernandoOFF requested a review from jkeyes November 7, 2025 13:58
@jkeyes
Copy link

jkeyes commented Nov 7, 2025

@FeernandoOFF I think this is a big improvement. I wonder why does each log message need to be rendered as it's own component? When you opened the share sheet it's so much easier to see all the information at a glance. I have an idea for a quick comparison of this. Why not make the background for the individual log entries the same as the container background (or vice-versa) and remove the padding between each individual entry. This would look more like a text editor basically, but maintain the colour highlighting which is useful for scanning. If you do that we can do a side by side comparison and see if I'm completely off base or not.

@FeernandoOFF
Copy link
Collaborator Author

Hi @jkeyes what do you think of this styling? I used just dividers to make it easier to read and give the hint that it can be interacted, and is not just informative text.

image

@jkeyes
Copy link

jkeyes commented Nov 10, 2025

what do you think of this styling? I used just dividers to make it easier to read and give the hint that it can be interacted, and is not just informative text.

Much easier to read now.

@FeernandoOFF FeernandoOFF merged commit 7a363cc into Tapadoo:main Nov 10, 2025
FeernandoOFF added a commit that referenced this pull request Nov 10, 2025
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.

2 participants