Skip to content

Fix z-index order of GPX overlay + color class refactor#438

Merged
JammingBen merged 2 commits into
opencloud-eu:mainfrom
Cassolette:patch/gpx-z-and-color-class-refactor
May 13, 2026
Merged

Fix z-index order of GPX overlay + color class refactor#438
JammingBen merged 2 commits into
opencloud-eu:mainfrom
Cassolette:patch/gpx-z-and-color-class-refactor

Conversation

@Cassolette
Copy link
Copy Markdown
Contributor

Description

1) Use oc color roles via generated tailwind utility classes opencloud-eu/web#2339

2) Fix z-index order of maps GPX overlay

Before/After:

image image

Related Issue

How Has This Been Tested?

  • test environment:
  • test case 1:
  • test case 2:
  • ...

Types of changes

  • Bugfix
  • Enhancement (a change that doesn't break existing code or deployments)
  • Breaking change (a modification that affects current functionality)
  • Technical debt (addressing code that needs refactoring or improvements)
  • Tests (adding or improving tests)
  • Documentation (updates or additions to documentation)
  • Maintenance (like dependency updates or tooling adjustments)

Cassolette added 2 commits May 8, 2026 19:11
extension-sdk has added support for these color role classes. Let's
switch to that instead of referencing the pre-existing global one from
design-system that Tailwind build system doesn't know about.
Ensure the GPX meta overlay stays below the app topbar (z-50).

Fixes # 380.
@Cassolette Cassolette marked this pull request as ready for review May 8, 2026 11:32
@Cassolette Cassolette force-pushed the patch/gpx-z-and-color-class-refactor branch from 9e4b60c to 263e0e8 Compare May 8, 2026 11:32
Copy link
Copy Markdown
Member

@JammingBen JammingBen left a comment

Choose a reason for hiding this comment

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

Thank you!

@JammingBen JammingBen merged commit 6a1fd53 into opencloud-eu:main May 13, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants