Skip to content

docs: Update log-with-tracing.md with trace_dbg! example - #529

Open
kdheepak wants to merge 1 commit into
mainfrom
kd/trace-dbg
Open

docs: Update log-with-tracing.md with trace_dbg! example#529
kdheepak wants to merge 1 commit into
mainfrom
kd/trace-dbg

Conversation

@kdheepak

Copy link
Copy Markdown
Member

No description provided.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying ratatui with  Cloudflare Pages  Cloudflare Pages

Latest commit: c82448f
Status: ✅  Deploy successful!
Preview URL: https://30dd7905.ratatui.pages.dev
Branch Preview URL: https://kd-trace-dbg.ratatui.pages.dev

View logs

@joshka joshka left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'd prefer to be consistent with the tracing-macros crate where this is found and use dbg! instead of trace_dbg!

Minimal preference for consistency. If you disagree based on trace_dbg being a generally better name, I'm happy to accept this as is.

@joshka

joshka commented Apr 1, 2024

Copy link
Copy Markdown
Member

I asked about the tracing_macros crate in the tokio discord: https://discord.com/channels/500028886025895936/974721015999516672/1224430040423334049

@kdheepak

kdheepak commented Apr 2, 2024

Copy link
Copy Markdown
Member Author

Happy to rename it! I didn’t realize that I swapped the names. Perhaps because I use dbg! so much normally.

@joshka

joshka commented Apr 2, 2024

Copy link
Copy Markdown
Member

Happy to rename it! I didn’t realize that I swapped the names. Perhaps because I use dbg! so much normally.

Ha, I assumed you'd done so with intention :D

@kdheepak

kdheepak commented Apr 2, 2024

Copy link
Copy Markdown
Member Author

If I did intend to do it I promptly forgot about it too :)

@joshka joshka mentioned this pull request Jun 4, 2024
joshka added a commit that referenced this pull request Jun 4, 2024
Adds a new `Block::border_set` method that allows the user to specify
the symbols used for the border.

Added two new border types: `BorderType::QuadrantOutside` and
`BorderType::QuadrantInside`. These are used to draw borders using the
unicode quadrant characters (which look like half block "pixels").

QuadrantOutside:
```
▛▀▀▜
▌  ▐
▙▄▄▟
```

QuadrantInside:
```
▗▄▄▖
▐  ▌
▝▀▀▘
```
Fixes: ratatui/ratatui#528

BREAKING CHANGES:
- BorderType::to_line_set is renamed to to_border_set
- BorderType::line_symbols is renamed to border_symbols
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