Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bring back theme switcher and remove default version picker #49892

Merged
merged 2 commits into from
Jan 23, 2025

Conversation

cristianjd
Copy link
Collaborator

@cristianjd cristianjd commented Jan 16, 2025

Looks like a sphynx update in verson 3.6.0 broke some of the layout this resolves the issues

  • adjusted the CSS to hide the default version picker that was covering the Ask AI button
  • moved the theme switched to navbar end so it shows up again
  • add logic so the version detection of the pydata sphynx picker works as expected for release/2.4.0 for example

Before:
Screenshot 2025-01-16 at 1 23 31 PM

After:
Screenshot 2025-01-16 at 1 23 09 PM

Checks

  • I've signed off every commit(by using the -s flag, i.e., git commit -s) in this PR.
  • I've run scripts/format.sh to lint the changes in this PR.
  • I've included any doc changes needed for https://docs.ray.io/en/master/.
    • I've added any new APIs to the API Reference. For example, if I added a
      method in Tune, I've added it in doc/source/tune/api/ under the
      corresponding .rst file.
  • I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failures at https://flakey-tests.ray.io/
  • Testing Strategy
    • Unit tests
    • Release tests
    • This PR is not tested :(

Sorry, something went wrong.

Signed-off-by: cristianjd <cristian.j.derr@gmail.com>
format
Signed-off-by: cristianjd <cristian.j.derr@gmail.com>
@cristianjd cristianjd requested a review from a team as a code owner January 16, 2025 21:34
Copy link
Contributor

@angelinalg angelinalg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

stamp

@cristianjd cristianjd added the go add ONLY when ready to merge, run all tests label Jan 22, 2025
@can-anyscale can-anyscale merged commit 1f5d734 into master Jan 23, 2025
5 checks passed
@can-anyscale can-anyscale deleted the cristian/layout-fixes branch January 23, 2025 16:15
anson627 pushed a commit to anson627/ray that referenced this pull request Jan 31, 2025
…ect#49892)

Looks like a sphynx update in verson 3.6.0 broke some of the layout this
resolves the issues
- adjusted the CSS to hide the default version picker that was covering
the Ask AI button
- moved the theme switched to navbar end so it shows up again
- add logic so the version detection of the pydata sphynx picker works
as expected for `release/2.4.0` for example

Before:
![Screenshot 2025-01-16 at 1 23
31 PM](https://github.com/user-attachments/assets/f877ba98-a753-41e9-b3b0-148575cd5904)

After:
![Screenshot 2025-01-16 at 1 23
09 PM](https://github.com/user-attachments/assets/4381904f-e1e7-4c09-bf5f-e393e99f62e3)

Signed-off-by: cristianjd <cristian.j.derr@gmail.com>
Signed-off-by: Anson Qian <anson627@gmail.com>
anson627 pushed a commit to anson627/ray that referenced this pull request Jan 31, 2025
…ect#49892)

Looks like a sphynx update in verson 3.6.0 broke some of the layout this
resolves the issues
- adjusted the CSS to hide the default version picker that was covering
the Ask AI button
- moved the theme switched to navbar end so it shows up again
- add logic so the version detection of the pydata sphynx picker works
as expected for `release/2.4.0` for example

Before:
![Screenshot 2025-01-16 at 1 23
31 PM](https://github.com/user-attachments/assets/f877ba98-a753-41e9-b3b0-148575cd5904)

After:
![Screenshot 2025-01-16 at 1 23
09 PM](https://github.com/user-attachments/assets/4381904f-e1e7-4c09-bf5f-e393e99f62e3)

Signed-off-by: cristianjd <cristian.j.derr@gmail.com>
Signed-off-by: Anson Qian <anson627@gmail.com>
srinathk10 pushed a commit that referenced this pull request Feb 2, 2025
Looks like a sphynx update in verson 3.6.0 broke some of the layout this
resolves the issues
- adjusted the CSS to hide the default version picker that was covering
the Ask AI button
- moved the theme switched to navbar end so it shows up again
- add logic so the version detection of the pydata sphynx picker works
as expected for `release/2.4.0` for example

Before:
![Screenshot 2025-01-16 at 1 23
31 PM](https://github.com/user-attachments/assets/f877ba98-a753-41e9-b3b0-148575cd5904)

After:
![Screenshot 2025-01-16 at 1 23
09 PM](https://github.com/user-attachments/assets/4381904f-e1e7-4c09-bf5f-e393e99f62e3)

Signed-off-by: cristianjd <cristian.j.derr@gmail.com>
cristianjd added a commit that referenced this pull request Feb 11, 2025
Looks like a sphynx update in verson 3.6.0 broke some of the layout this
resolves the issues
- adjusted the CSS to hide the default version picker that was covering
the Ask AI button
- moved the theme switched to navbar end so it shows up again
- add logic so the version detection of the pydata sphynx picker works
as expected for `release/2.4.0` for example

Before:
![Screenshot 2025-01-16 at 1 23
31 PM](https://github.com/user-attachments/assets/f877ba98-a753-41e9-b3b0-148575cd5904)

After:
![Screenshot 2025-01-16 at 1 23
09 PM](https://github.com/user-attachments/assets/4381904f-e1e7-4c09-bf5f-e393e99f62e3)

Signed-off-by: cristianjd <cristian.j.derr@gmail.com>
cristianjd added a commit that referenced this pull request Feb 11, 2025
Looks like a sphynx update in verson 3.6.0 broke some of the layout this
resolves the issues
- adjusted the CSS to hide the default version picker that was covering
the Ask AI button
- moved the theme switched to navbar end so it shows up again
- add logic so the version detection of the pydata sphynx picker works
as expected for `release/2.4.0` for example

Before:
![Screenshot 2025-01-16 at 1 23
31 PM](https://github.com/user-attachments/assets/f877ba98-a753-41e9-b3b0-148575cd5904)

After:
![Screenshot 2025-01-16 at 1 23
09 PM](https://github.com/user-attachments/assets/4381904f-e1e7-4c09-bf5f-e393e99f62e3)

Signed-off-by: cristianjd <cristian.j.derr@gmail.com>
cristianjd added a commit that referenced this pull request Feb 11, 2025
Looks like a sphynx update in verson 3.6.0 broke some of the layout this
resolves the issues
- adjusted the CSS to hide the default version picker that was covering
the Ask AI button
- moved the theme switched to navbar end so it shows up again
- add logic so the version detection of the pydata sphynx picker works
as expected for `release/2.4.0` for example

Before:
![Screenshot 2025-01-16 at 1 23
31 PM](https://github.com/user-attachments/assets/f877ba98-a753-41e9-b3b0-148575cd5904)

After:
![Screenshot 2025-01-16 at 1 23
09 PM](https://github.com/user-attachments/assets/4381904f-e1e7-4c09-bf5f-e393e99f62e3)

Signed-off-by: cristianjd <cristian.j.derr@gmail.com>
cristianjd added a commit that referenced this pull request Feb 11, 2025
Looks like a sphynx update in verson 3.6.0 broke some of the layout this
resolves the issues
- adjusted the CSS to hide the default version picker that was covering
the Ask AI button
- moved the theme switched to navbar end so it shows up again
- add logic so the version detection of the pydata sphynx picker works
as expected for `release/2.4.0` for example

Before:
![Screenshot 2025-01-16 at 1 23
31 PM](https://github.com/user-attachments/assets/f877ba98-a753-41e9-b3b0-148575cd5904)

After:
![Screenshot 2025-01-16 at 1 23
09 PM](https://github.com/user-attachments/assets/4381904f-e1e7-4c09-bf5f-e393e99f62e3)

Signed-off-by: cristianjd <cristian.j.derr@gmail.com>
cristianjd added a commit that referenced this pull request Feb 11, 2025
Looks like a sphynx update in verson 3.6.0 broke some of the layout this
resolves the issues
- adjusted the CSS to hide the default version picker that was covering
the Ask AI button
- moved the theme switched to navbar end so it shows up again
- add logic so the version detection of the pydata sphynx picker works
as expected for `release/2.4.0` for example

Before:
![Screenshot 2025-01-16 at 1 23
31 PM](https://github.com/user-attachments/assets/f877ba98-a753-41e9-b3b0-148575cd5904)

After:
![Screenshot 2025-01-16 at 1 23
09 PM](https://github.com/user-attachments/assets/4381904f-e1e7-4c09-bf5f-e393e99f62e3)

Signed-off-by: cristianjd <cristian.j.derr@gmail.com>
cristianjd added a commit that referenced this pull request Feb 11, 2025
Looks like a sphynx update in verson 3.6.0 broke some of the layout this
resolves the issues
- adjusted the CSS to hide the default version picker that was covering
the Ask AI button
- moved the theme switched to navbar end so it shows up again
- add logic so the version detection of the pydata sphynx picker works
as expected for `release/2.4.0` for example

Before:
![Screenshot 2025-01-16 at 1 23
31 PM](https://github.com/user-attachments/assets/f877ba98-a753-41e9-b3b0-148575cd5904)

After:
![Screenshot 2025-01-16 at 1 23
09 PM](https://github.com/user-attachments/assets/4381904f-e1e7-4c09-bf5f-e393e99f62e3)

Signed-off-by: cristianjd <cristian.j.derr@gmail.com>
cristianjd added a commit that referenced this pull request Feb 11, 2025
Looks like a sphynx update in verson 3.6.0 broke some of the layout this
resolves the issues
- adjusted the CSS to hide the default version picker that was covering
the Ask AI button
- moved the theme switched to navbar end so it shows up again
- add logic so the version detection of the pydata sphynx picker works
as expected for `release/2.4.0` for example

Before:
![Screenshot 2025-01-16 at 1 23
31 PM](https://github.com/user-attachments/assets/f877ba98-a753-41e9-b3b0-148575cd5904)

After:
![Screenshot 2025-01-16 at 1 23
09 PM](https://github.com/user-attachments/assets/4381904f-e1e7-4c09-bf5f-e393e99f62e3)

Signed-off-by: cristianjd <cristian.j.derr@gmail.com>
pcmoritz pushed a commit that referenced this pull request Feb 11, 2025
Cherry picks the following PRs to version 2.41.0 to fix version switcher
issues

#49892
#50292

## Checks

- [ ] I've signed off every commit(by using the -s flag, i.e., `git
commit -s`) in this PR.
- [ ] I've run `scripts/format.sh` to lint the changes in this PR.
- [ ] I've included any doc changes needed for
https://docs.ray.io/en/master/.
- [ ] I've added any new APIs to the API Reference. For example, if I
added a
method in Tune, I've added it in `doc/source/tune/api/` under the
           corresponding `.rst` file.
- [ ] I've made sure the tests are passing. Note that there might be a
few flaky tests, see the recent failures at https://flakey-tests.ray.io/
- Testing Strategy
   - [ ] Unit tests
   - [ ] Release tests
   - [ ] This PR is not tested :(

---------

Signed-off-by: cristianjd <cristian.j.derr@gmail.com>
pcmoritz pushed a commit that referenced this pull request Feb 13, 2025
Cherry picks the following PRs to version 2.39.0 to fix version switcher
issues

#49892
#50292

## Checks

- [ ] I've signed off every commit(by using the -s flag, i.e., `git
commit -s`) in this PR.
- [ ] I've run `scripts/format.sh` to lint the changes in this PR.
- [ ] I've included any doc changes needed for
https://docs.ray.io/en/master/.
- [ ] I've added any new APIs to the API Reference. For example, if I
added a
method in Tune, I've added it in `doc/source/tune/api/` under the
           corresponding `.rst` file.
- [ ] I've made sure the tests are passing. Note that there might be a
few flaky tests, see the recent failures at https://flakey-tests.ray.io/
- Testing Strategy
   - [ ] Unit tests
   - [ ] Release tests
   - [ ] This PR is not tested :(

---------

Signed-off-by: cristianjd <cristian.j.derr@gmail.com>
anyadontfly pushed a commit to anyadontfly/ray that referenced this pull request Feb 13, 2025
…ect#49892)

Looks like a sphynx update in verson 3.6.0 broke some of the layout this
resolves the issues
- adjusted the CSS to hide the default version picker that was covering
the Ask AI button
- moved the theme switched to navbar end so it shows up again
- add logic so the version detection of the pydata sphynx picker works
as expected for `release/2.4.0` for example

Before:
![Screenshot 2025-01-16 at 1 23
31 PM](https://github.com/user-attachments/assets/f877ba98-a753-41e9-b3b0-148575cd5904)

After:
![Screenshot 2025-01-16 at 1 23
09 PM](https://github.com/user-attachments/assets/4381904f-e1e7-4c09-bf5f-e393e99f62e3)

Signed-off-by: cristianjd <cristian.j.derr@gmail.com>
Signed-off-by: Puyuan Yao <williamyao034@gmail.com>
park12sj pushed a commit to park12sj/ray that referenced this pull request Mar 18, 2025
…ect#49892)

Looks like a sphynx update in verson 3.6.0 broke some of the layout this
resolves the issues
- adjusted the CSS to hide the default version picker that was covering
the Ask AI button
- moved the theme switched to navbar end so it shows up again
- add logic so the version detection of the pydata sphynx picker works
as expected for `release/2.4.0` for example

Before:
![Screenshot 2025-01-16 at 1 23
31 PM](https://github.com/user-attachments/assets/f877ba98-a753-41e9-b3b0-148575cd5904)

After:
![Screenshot 2025-01-16 at 1 23
09 PM](https://github.com/user-attachments/assets/4381904f-e1e7-4c09-bf5f-e393e99f62e3)

Signed-off-by: cristianjd <cristian.j.derr@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
go add ONLY when ready to merge, run all tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants