Skip to content

fix(styling): always keep active cell visible during row hover#2461

Merged
ghiscoding merged 1 commit intomasterfrom
bugfix/salesforce-active-cell
Mar 13, 2026
Merged

fix(styling): always keep active cell visible during row hover#2461
ghiscoding merged 1 commit intomasterfrom
bugfix/salesforce-active-cell

Conversation

@ghiscoding
Copy link
Owner

@ghiscoding ghiscoding commented Mar 13, 2026

supersede PR #2454

it's an issue only in the Salesforce Theme.

I always had an issue with the active cell look & feel, if we were hovering the row then it would cover the active cell border. What I really want is to always have the active cell on top of all other styling. That was a very small (but annoying to me) visual issue that bothered me for a while but I just didn't know how to fix it, so thanks to copilot for figuring out a fix, now the UX is looking so much better 🙈

before the fix

msedge_MMgRkP5GF4

after the fix (the active cell with blue border is always on top as it should be)

msedge_gkVIj5Vk5w

@ghiscoding ghiscoding changed the title fix(styling): alwyas keep active cell visible during row hover fix(styling): always keep active cell visible during row hover Mar 13, 2026
@codecov
Copy link

codecov bot commented Mar 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.0%. Comparing base (2b31157) to head (62458ab).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2461   +/-   ##
=======================================
  Coverage   100.0%   100.0%           
=======================================
  Files         195      195           
  Lines       24402    24402           
  Branches     8570     8570           
=======================================
  Hits        24402    24402           
Flag Coverage Δ
angular 100.0% <ø> (ø)
universal 100.0% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 13, 2026

angular-slickgrid

npm i https://pkg.pr.new/angular-slickgrid@2461

aurelia-slickgrid

npm i https://pkg.pr.new/aurelia-slickgrid@2461

slickgrid-react

npm i https://pkg.pr.new/slickgrid-react@2461

slickgrid-vue

npm i https://pkg.pr.new/slickgrid-vue@2461

@slickgrid-universal/angular-row-detail-plugin

npm i https://pkg.pr.new/@slickgrid-universal/angular-row-detail-plugin@2461

@slickgrid-universal/aurelia-row-detail-plugin

npm i https://pkg.pr.new/@slickgrid-universal/aurelia-row-detail-plugin@2461

@slickgrid-universal/react-row-detail-plugin

npm i https://pkg.pr.new/@slickgrid-universal/react-row-detail-plugin@2461

@slickgrid-universal/vue-row-detail-plugin

npm i https://pkg.pr.new/@slickgrid-universal/vue-row-detail-plugin@2461

@slickgrid-universal/binding

npm i https://pkg.pr.new/@slickgrid-universal/binding@2461

@slickgrid-universal/common

npm i https://pkg.pr.new/@slickgrid-universal/common@2461

@slickgrid-universal/composite-editor-component

npm i https://pkg.pr.new/@slickgrid-universal/composite-editor-component@2461

@slickgrid-universal/custom-footer-component

npm i https://pkg.pr.new/@slickgrid-universal/custom-footer-component@2461

@slickgrid-universal/custom-tooltip-plugin

npm i https://pkg.pr.new/@slickgrid-universal/custom-tooltip-plugin@2461

@slickgrid-universal/empty-warning-component

npm i https://pkg.pr.new/@slickgrid-universal/empty-warning-component@2461

@slickgrid-universal/event-pub-sub

npm i https://pkg.pr.new/@slickgrid-universal/event-pub-sub@2461

@slickgrid-universal/excel-export

npm i https://pkg.pr.new/@slickgrid-universal/excel-export@2461

@slickgrid-universal/graphql

npm i https://pkg.pr.new/@slickgrid-universal/graphql@2461

@slickgrid-universal/odata

npm i https://pkg.pr.new/@slickgrid-universal/odata@2461

@slickgrid-universal/pagination-component

npm i https://pkg.pr.new/@slickgrid-universal/pagination-component@2461

@slickgrid-universal/pdf-export

npm i https://pkg.pr.new/@slickgrid-universal/pdf-export@2461

@slickgrid-universal/row-detail-view-plugin

npm i https://pkg.pr.new/@slickgrid-universal/row-detail-view-plugin@2461

@slickgrid-universal/rxjs-observable

npm i https://pkg.pr.new/@slickgrid-universal/rxjs-observable@2461

@slickgrid-universal/text-export

npm i https://pkg.pr.new/@slickgrid-universal/text-export@2461

@slickgrid-universal/utils

npm i https://pkg.pr.new/@slickgrid-universal/utils@2461

@slickgrid-universal/vanilla-bundle

npm i https://pkg.pr.new/@slickgrid-universal/vanilla-bundle@2461

@slickgrid-universal/vanilla-force-bundle

npm i https://pkg.pr.new/@slickgrid-universal/vanilla-force-bundle@2461

commit: 7a8510f

@ghiscoding ghiscoding merged commit 5ed45aa into master Mar 13, 2026
20 checks passed
@ghiscoding ghiscoding deleted the bugfix/salesforce-active-cell branch March 13, 2026 22:12
@github-actions
Copy link

🎉 This pull request is included in version 10.2.0 📦
🔗 The release notes are available at: GitHub Release 🚀

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.

1 participant