changed padding values and added borderRadius

[Kritarthsharma]

Fixes https://github.com/AppFlowy-IO/AppFlowy/issues/3313

• Modified the appflowy_editor package to implement the requested UI fix.
• Changed the padding values and added borderRadius for more consistent look.

Screen.Recording.2023-10-30.at.5.58.17.PM.mov

[CLAassistant]

All committers have signed the CLA.

[MayurSMahajan]

Hey @Kritarthsharma,

actually, you were not supposed to add a border radius on the option, but the container which displays the options. Basically when there are no results the container has less border radius.

Also, I see that you are padding to the no-results text. Rather than applying padding to the No Results text, you can calculate the width of other options used in the Selection Menu and then use that fixed width with center-aligned text. See if that is possible.

[Kritarthsharma]

Hi @MayurSMahajan, I've made changes as you suggested. Can you please review it?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (b8368d8) 80.43% compared to head (6fb0f04) 80.43%.
Report is 4 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #567   +/-   ##
=======================================
  Coverage   80.43%   80.43%           
=======================================
  Files         285      285           
  Lines       12194    12194           
=======================================
  Hits         9808     9808           
  Misses       2386     2386           

Files	Coverage Δ
...c/editor/selection_menu/selection_menu_widget.dart	40.82% <ø> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.