Skip to content

Scix 687 highlight update#740

Merged
thostetler merged 2 commits intoadsabs:masterfrom
shinyichen:SCIX-687-highlight-update
Dec 19, 2025
Merged

Scix 687 highlight update#740
thostetler merged 2 commits intoadsabs:masterfrom
shinyichen:SCIX-687-highlight-update

Conversation

@shinyichen
Copy link
Member

@shinyichen shinyichen commented Dec 17, 2025

Depends on #739

  • Move the toggle out from the 'settings' menu
  • Show where the highlight is from (abstract, title ..)
Screenshot 2025-12-17 at 2 58 30 PM

@codecov
Copy link

codecov bot commented Dec 17, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.7%. Comparing base (1ba99d5) to head (c44b735).

Additional details and impacted files
@@           Coverage Diff            @@
##           master    #740     +/-   ##
========================================
+ Coverage    69.7%   69.7%   +0.1%     
========================================
  Files         219     220      +1     
  Lines       24513   24562     +49     
  Branches     1304    1302      -2     
========================================
+ Hits        17066   17103     +37     
- Misses       7412    7424     +12     
  Partials       35      35             

see 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@shinyichen shinyichen force-pushed the SCIX-687-highlight-update branch from fa7ac87 to c44b735 Compare December 18, 2025 18:52
@thostetler thostetler force-pushed the SCIX-687-highlight-update branch from c44b735 to 01713ff Compare December 19, 2025 14:50
Copy link
Member

@thostetler thostetler left a comment

Choose a reason for hiding this comment

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

Changes LGTM

@thostetler thostetler merged commit 3311a34 into adsabs:master Dec 19, 2025
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