-
Notifications
You must be signed in to change notification settings - Fork 85
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
Activating edge matching on the selected feature even when no feature in other track aligns with the selected feature is confusing. #9
Comments
I realised I don't have a working WA setup, so won't be able to send a (tested) PR. I am sorry. The patch above is tested on https://github.com/yeban/afra. |
edge-matches with it. GMOD/Apollo#9 Signed-off-by: Anurag Priyam <[email protected]>
I don't see how this is "confusing because a user will be tempted to scroll down to look for corresponding edge-matched feature and find none". The selected feature is highlighted and it overrides the edge-matching so there shouldn't be any confusion there. When selecting a whole feature (double clicking), having the subfeatures selected helps enforce the fact that everything is selected. |
When selecting a whole feature, having the subfeatures selected is redundant, imo. When I see a border enclosing 3 rectangles (exons), I intuitively conclude all 3 are selected and not an imaginary outer container. |
Need to review this one again. |
After implementing, the change is so slight, I don't think that most users could even tell, but I will leave the diff here for posterity. |
Actually, I can see the reason for doing this. At some point a separate color for matches versus highglightin might be better, though. |
So I select a feature and its edges do not match with any other feature in all tracks displayed. Yet,
left-edge-match
andright-edge-match
classes are applied to the selected feature. This is confusing because a user will be tempted to scroll down to look for corresponding edge-matched feature and find none.This happens because FeatureEdgeManager iterates through all tracks including the one containing the selected feature and thus matches the selected feature with itself. A feature will always edge-match on to itself.
Following rough patch (git diff) is a fix. I can send a PR if you agree.
The text was updated successfully, but these errors were encountered: