Multibyte characters will be displayed incorrectly.

[imakado]

Multibyte characters will be displayed incorrectly.

[meliache]

With emacs 29 getting full emoji support I decoreated some of my hydra heads with emoji, which looks really nice but also breaks the alignment. E.g. with the current pretty-hydra the following hydra

  (pretty-hydra-define org-clock-hydra
    (:color blue :title "Org Clock ⏱️️")
    ("Clock"
     (("i" org-clock-in "in ")
      ("o" org-clock-out "out ")
      ("l" org-clock-in-last "in last")
      ("s" my/org-clock-in-select "select")
      ("t" org-clock-hydra-into-time-sink/body "timesink")
      ("p" org-pomodoro "pomodoro 🍅"))
     "Edit"
     (("e" org-clock-modify-effort-estimate "set effort 🏋🏻‍♀️")
      ("x" org-pomodoro-extend-last-clock "extend last")
      ("c" org-clock-cancel "cancel clock 🚫")
      ("v" #'org-resolve-clocks  "resolve"))
     "Summary"
     (("g" org-clock-goto "goto clock 🚶🏾‍♀️")
      ("d" org-clock-display "display 🖥️")
      ("r" org-clock-report "report 📜")
      ("?" (org-info "Clocking commands") "info"))))

looks like the following:

With the fix from the comment by @imakado I get the following result:

Looks like I might also need to look into the code more and check where else equality of length and width is assumed.

[imakado]

Sorry for the crappy report. I'm still reading the source code...

[jerrypnz]

Thanks for reporting this. I don't personally use CJK characters in my setup but I understand it's annoying when columns don't align properly.
Do you think if you can raise a pull request for your suggested change?

[jerrypnz]

I have a feeling it highly depends on the font. I'll look into it a bit more.

[meliache]

I was quite active on the issues here and initiated some PR's but also disappeared for a bit. The main reason being that I switchend to Transient for menus with automatic layouts. The recent Transient versions have the option transient-align-variable-pitch, which implements pixel-alignment:
https://github.com/magit/transient/blob/04f87b4ab3c7448a78f452cadaaf5d9e7a216d1d/lisp/transient.el#L315-L330
Not sure is what @imakado suggested above does the same. If interested you might look into the transient code for inspiration. In transient the option is nil by default because of the associated performance penalty.