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

Add tooltips in jobs columns #362

Merged
merged 1 commit into from
May 31, 2024
Merged

Conversation

brianlove
Copy link
Contributor

@brianlove brianlove commented May 23, 2024

Add tooltips in the jobs columns providing explanatory information about limitations of the numbers.

Closes #326

@brianlove brianlove self-assigned this May 23, 2024
Copy link

No need for rebasing 👍
behind_count is 0
ahead_count is 1

Copy link

github-actions bot commented May 23, 2024

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
594 423 71% 0% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: 2b7107a by action🐍

Copy link

github-actions bot commented May 23, 2024

JavaScript Coverage

Summary

Lines Statements Branches Functions
Coverage: 67%
67.91% (398/586) 55.34% (207/374) 68.1% (126/185)
Modified Files • (67%)
File% Stmts% Branch% Funcs% LinesUncovered Line #s
All files67.9155.3468.167.69 
components64.7555.8662.5964.33 
   AddRemoveColumnDialog.jsx91.667583.3391.379, 99
   CellStat.jsx10085.7110010063, 75–78
   ListViewTable.jsx91.0173.8398.3690.9286, 369–370, 375–378, 487, 532, 692, 713–716, 732–733, 757–758, 761, 779–782, 802–807
static_data67.8555.1777.2765.38 
   table_columns.js66.6655.1777.2764223–576
   tooltips.js100100100100 

@brianlove brianlove force-pushed the 326-tooltips-for-jobs-numbers branch from a531cc9 to 6bcead9 Compare May 30, 2024 22:06
@brianlove brianlove changed the title WIP: Add tooltips in jobs columns Add tooltips in jobs columns May 30, 2024
@brianlove brianlove marked this pull request as ready for review May 30, 2024 22:10
@brianlove brianlove requested a review from jmelot May 30, 2024 22:11
Add tooltips in the jobs columns providing explanatory information
about limitations of the numbers.  Update uses of HelpTooltip to
use small icons where appropriate.

Closes #326
@jmelot jmelot force-pushed the 326-tooltips-for-jobs-numbers branch from 6bcead9 to 2b7107a Compare May 31, 2024 13:06
Copy link
Member

@jmelot jmelot left a comment

Choose a reason for hiding this comment

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

Looks great!

country = undefined,
data,
}) => {
const tooltipType = (country === "United States") ? undefined : (
Copy link
Member

Choose a reason for hiding this comment

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

This is a little bit hard to read, might be easier as a series of if statements!

@jmelot jmelot merged commit 309f0dd into master May 31, 2024
6 checks passed
@jmelot jmelot deleted the 326-tooltips-for-jobs-numbers branch May 31, 2024 13:58
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.

alert next to Chinese jobs numbers?
2 participants