search

rcarriga / nvim-notify

A fancy, configurable, notification manager for NeoVim
MIT License
3.09k stars 83 forks source link

Notification width does not take title length into account #68

Closed vogelsgesang closed 2 years ago

vogelsgesang commented 2 years ago

The width used to render a notification takes only the body length into account, but not the title widht. This means that the messages by LSP or DAP (where titles can easily be longer than the body) sometimes render badly. For example,

require("notify")("Complete", "info", {title = "lldb: Parsing symbol table for test_binary", timeout = 3000})

renders as

Screenshot 2022-03-14 at 10 41 16
rcarriga commented 2 years ago

Thanks for reporting, should be taking titles into account now :grin:

vogelsgesang commented 2 years ago

works beautifully! Thanks :)