Issue 536: Points across peak count should scale with chromatogram graph font settings

issues
Status:closed
Assigned To:Guest
Type:Defect
Area:Skyline
Priority:4
Milestone:4.1
Opened:2017-08-13 by Brendan MacLean
Changed:2017-08-31 by Brendan MacLean
Resolved:2017-08-14 by Yuval
Resolution:Fixed
Closed:2017-08-31 by Brendan MacLean
2017-08-13 Brendan MacLean
Title»Points across peak count should scale with chromatogram graph font settings
Assigned To»Yuval
Type»Defect
Area»Skyline
Priority»4
Milestone»4.1
The new "Show Raw Times" feature in the chromatograms graph shows the number of time points between the integration boundaries, but that number does not change size based on the font sizing setting in the chromatogram graph, as all other annotations do.

2017-08-14 Yuval
resolve as Fixed
Statusopen»resolved
Assigned ToYuval»Brendan MacLean
Resolved with revision 11210

2017-08-14 Brendan MacLean
reopen
Statusresolved»open
Assigned ToBrendan MacLean»Yuval
Not exactly the same issue, but while you are on this, I think it would be good to show a question mark "?" where you show the number when raw times are not available. I found it very confusing to just get nothing. I took me a while to realize it was because I had opened an older file. I first turned on and off the menu item and restarted Skyline and was even about to start debugging, before I realized why the lines and number were not showing up.

Putting a question mark there will make it way more obvious and likely save us on support costs, because if they see a question mark, we don't have to double-check that they have the right version of Skyline with support for this feature. Otherwise, the problem is ambiguous. Is it (1) the Skyline version being used doesn't have the feature (2) the feature is not working for a valid file (3) the file does not contain raw times.

2017-08-14 Yuval
resolve as Fixed
Statusopen»resolved
Assigned ToYuval»Brendan MacLean
Fixed in rev 11212

2017-08-31 Brendan MacLean
close
Statusresolved»closed
Assigned ToBrendan MacLean»Guest