## Describe your changes

Use a different mechanism using `onUpdated` callback to keep the Plotly
state.

## GitHub Issue Link (if applicable)

## Testing Plan

- Explanation of why no additional tests are needed
- Unit Tests (JS and/or Python)
- E2E Tests
- Any manual testing needed?

---

**Contribution License Agreement**

By submitting this pull request you agree that all contributions to this
project are made under the Apache 2.0 license.

---------

Co-authored-by: William Wei Huang <william.huang@snowflake.com>

<!--
⚠️ BEFORE CONTRIBUTING PLEASE READ OUR CONTRIBUTING GUIDELINES!
https://github.com/streamlit/streamlit/wiki/Contributing
-->

## Describe your changes
- refactor removing boolean because "rerun" and "ignore" are the
preferred ways of setting `on_select`
- should likely wait to get this PR in until other PRs such as
#8522 and
#8515 are merged
## GitHub Issue Link (if applicable)

## Testing Plan

- Explanation of why no additional tests are needed
- Unit Tests (JS and/or Python)
- E2E Tests
- Any manual testing needed?

---

**Contribution License Agreement**

By submitting this pull request you agree that all contributions to this
project are made under the Apache 2.0 license.

---------

Co-authored-by: lukasmasuch <lukas.masuch@gmail.com>

…iveCharts_Plotly

…ctiveCharts_Plotly' of https://github.com/streamlit/streamlit into feature/InteractiveCharts_Plotly

…/streamlit/streamlit into feature/InteractiveCharts_Plotly

…iveCharts_Plotly

## Describe your changes
- best attempt to fix typechecking for plotly
- there is one # type: ignore at line 185 because mypy is not
recognizing that they're compatible sets
## GitHub Issue Link (if applicable)

## Testing Plan

- Explanation of why no additional tests are needed
- Unit Tests (JS and/or Python)
- E2E Tests
- Any manual testing needed?

---

**Contribution License Agreement**

By submitting this pull request you agree that all contributions to this
project are made under the Apache 2.0 license.

## Describe your changes
- fix python tests
## GitHub Issue Link (if applicable)

## Testing Plan

- Explanation of why no additional tests are needed
- Unit Tests (JS and/or Python)
- E2E Tests
- Any manual testing needed?

---

**Contribution License Agreement**

By submitting this pull request you agree that all contributions to this
project are made under the Apache 2.0 license.

---------

Co-authored-by: Lukas Masuch <Lukas.Masuch@gmail.com>

## Describe your changes
- new fullscreen changes have come in so need to fix the playwright code
to look for new full screen / exit full screen buttons
- new fullscreen changes so snapshots likely need to be regenerated
## GitHub Issue Link (if applicable)

## Testing Plan

- Explanation of why no additional tests are needed
- Unit Tests (JS and/or Python)
- E2E Tests
- Any manual testing needed?

---

**Contribution License Agreement**

By submitting this pull request you agree that all contributions to this
project are made under the Apache 2.0 license.

---------

Co-authored-by: Lukas Masuch <Lukas.Masuch@gmail.com>

…/streamlit/streamlit into feature/InteractiveCharts_Plotly

## Describe your changes
- title
- added tests for:
  - `sendEmptySelection`
  - `handleSelection`
  - `applyTheming`
## GitHub Issue Link (if applicable)

## Testing Plan

- Explanation of why no additional tests are needed
- Unit Tests (JS and/or Python)
- E2E Tests
- Any manual testing needed?

---

**Contribution License Agreement**

By submitting this pull request you agree that all contributions to this
project are made under the Apache 2.0 license.

## Describe your changes
- null was being passed to resetSelectionInFigure because Plotly will
pass in null. As a result, we need to actually call the function ourself
to get the default value
- doing this also changes the default plotly pan behavior and also
resets selections too
Current:


https://github.com/streamlit/streamlit/assets/16749069/cb14a994-7d0f-4425-9c4f-a660ae9248bf


Now:


https://github.com/streamlit/streamlit/assets/16749069/8820f163-4294-4781-b12d-04d6a750ab12


## GitHub Issue Link (if applicable)

## Testing Plan

- Explanation of why no additional tests are needed
- Unit Tests (JS and/or Python)
- E2E Tests
- Any manual testing needed?

---

**Contribution License Agreement**

By submitting this pull request you agree that all contributions to this
project are made under the Apache 2.0 license.

…iveCharts_Altair

…wasn't working (#8570)

This reverts commit 66983ce.