refactor: remove icons from UI

[jimniels]

IMO we have too many icons. It's diluting the value/efficacy of the ones we do have. I think we should use them a lot more sparingly. My impulse is to remove a bunch, and then we can always add them back where we can justify their value.

The new command palette

which is a lot more akin to Figma's: graphical elements are only used for 1) toggles, to indicate the on/off state of an item, or 2) external links, indicating these items will take you out of the app.

(Figma's UI)

I also removed them from dropdown menus

For now, I think it's ok to keep them in the formatting menu.

Todos

• Once feat: export to CSV #604 is done, remove the icons from the floating flyout menu and then merge this

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
quadratic	✅ Ready (Inspect)	Visit Preview	Nov 17, 2023 5:50pm

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (bdf6e23) 82.76% compared to head (2ed5fcd) 82.76%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #826   +/-   ##
=======================================
  Coverage   82.76%   82.76%           
=======================================
  Files         116      116           
  Lines       17339    17339           
=======================================
  Hits        14350    14350           
  Misses       2989     2989           

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

[ddimaria]

You've been cursed by the single commit "feature" in Heroku

[davidkircos]

For me it's easier to skim for symbols than it is to read each item. Esp for formatting items icons make a lot of sense to me.

Why do you want to remove them?

[jimniels]

This is going to be superceeded by the work in #840