Skip to content

Conversation

@AlexanderLanin
Copy link
Member

Yet another attempt of making version switcher readable in the new purple style.

@github-actions
Copy link

github-actions bot commented Sep 15, 2025

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run //src:license-check

Status: ⚠️ Needs Review

Click to expand output
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.3.0) and connecting to it...
INFO: Invocation ID: 071d44d6-036c-4dcc-98ae-89392a2d3aea
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: src
Analyzing: target //src:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //src:license-check (1 packages loaded, 0 targets configured)

Analyzing: target //src:license-check (70 packages loaded, 9 targets configured)

Analyzing: target //src:license-check (73 packages loaded, 9 targets configured)

Analyzing: target //src:license-check (123 packages loaded, 1034 targets configured)

Analyzing: target //src:license-check (129 packages loaded, 2435 targets configured)

Analyzing: target //src:license-check (129 packages loaded, 2435 targets configured)

INFO: Analyzed target //src:license-check (137 packages loaded, 4500 targets configured).
[8 / 13] Creating runfiles tree bazel-out/k8-opt-exec-ST-d57f47055a04/bin/external/score_tooling+/dash/tool/formatters/dash_format_converter.runfiles [for tool]; 0s local ... (2 actions, 1 running)
INFO: Found 1 target...
Target //src:license.check.license_check up-to-date:
  bazel-bin/src/license.check.license_check
  bazel-bin/src/license.check.license_check.jar
INFO: Elapsed time: 14.466s, Critical Path: 0.38s
INFO: 13 processes: 4 disk cache hit, 9 internal.
INFO: Build completed successfully, 13 total actions
INFO: Running command line: bazel-bin/src/license.check.license_check src/formatted.txt <args omitted>
usage: org.eclipse.dash.licenses.cli.Main [-batch <int>] [-cd <url>]
       [-confidence <int>] [-ef <url>] [-excludeSources <sources>] [-help] [-lic
       <url>] [-project <shortname>] [-repo <url>] [-review] [-summary <file>]
       [-timeout <seconds>] [-token <token>]

@github-actions
Copy link

The created documentation from the pull request is available at: docu-html

@AlexanderLanin AlexanderLanin marked this pull request as ready for review September 15, 2025 17:43
Copilot AI review requested due to automatic review settings September 15, 2025 17:43
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes styling issues with the version switcher component in the purple theme by updating CSS selectors and improving color contrast for better readability.

  • Replaces ineffective version switcher container styling with targeted button and menu styling
  • Adds explicit white text color for version switcher button and menu items
  • Removes TODO comment indicating the previous approach wasn't working
Comments suppressed due to low confidence (1)

src/extensions/score_layout/assets/css/score.css:10

  • Typo in comment: 'colro' should be 'color'.
    A list of available colro variable names for pyData Sphinx Theme can be found at

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@AlexanderLanin
Copy link
Member Author

It works! Thanks @MaximilianSoerenPollak !

@AlexanderLanin AlexanderLanin merged commit 21486a8 into eclipse-score:main Sep 15, 2025
11 checks passed
@github-project-automation github-project-automation bot moved this from Draft to Done in Infrastructure Sep 15, 2025
@AlexanderLanin AlexanderLanin deleted the style branch September 15, 2025 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants