[Tidy] Remove CSS classes to position image

[huong-li-nguyen]

Description

• Remove custom CSS classes in static folder for positioning images (see comments below): [Tidy] Remove CSS classes to position image #919 (comment)

@stichbury - could you take a look at the docs changes for me? 🙏

Screenshot

Notice

• I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

  • I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.
  • I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorized to submit this contribution on behalf of the original creator(s) or their licensees.
  • I certify that the use of this contribution as authorized by the Apache 2.0 license does not violate the intellectual property rights of anyone else.
  • I have not referenced individuals, products or companies in any commits, directly or indirectly.
  • I have not added data or restricted code in any commits, directly or indirectly.

[github-actions]

View the example dashboards of the current commit live on PyCafe ☕ 🚀

Updated on: 2024-12-04 16:39:22 UTC
Commit: c37c2ae

Link: vizro-core/examples/dev/

Link: vizro-core/examples/scratch_dev

Link: vizro-core/examples/visual-vocabulary/

Link: vizro-ai/examples/dashboard_ui/

[stichbury]

Just one query @huong-li-nguyen

I'm seeing a warning in Py.Cafe for the second example: https://vizro--919.org.readthedocs.build/en/919/pages/tutorials/explore-components/#22-add-further-components in the built docs:

Using the title argument in your Plotly chart function may cause misalignment with other component titles on the screen. To ensure consistent alignment, consider using vm.Graph(title='Title', ...).

This isn't code you've changed, but would you mind checking if this is anything that we need to remedy please?

Otherwise, changes LGTM 🏆 🥇

[huong-li-nguyen]

This isn't code you've changed, but would you mind checking if this is anything that we need to remedy please?

Fixed :) Not sure why we had the argument title="" in there, because there is not title by default 🤔 Anyway, in case we do add a title to a chart we would do it like this now: vm.Graph(title="XXX", figure=px.box(...)) instead of this vm.Graph(figure=px.box(title="XXX"))

If someone uses the second approach, a warning will be raised as the titles might be misaligned with other components