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 ability to specify "multiple highlights" or bookmarks #668

Merged
merged 22 commits into from
Feb 23, 2016

Conversation

cmdcolin
Copy link
Contributor

@cmdcolin cmdcolin commented Dec 8, 2015

This adds a couple functions to essentially have "multiple highlights". The highlights can have labels and can be different colors (maybe they are like bookmarks). You can define the bookmarks in a config file, on the URL bar, or via a web service.

I think originally the idea of multiple highlights was brought up here #482

Each bookmark has

start (int), end (int), ref (string)

Optional params

color (string for css compatible color)
label (string for label on left side)
rlabel (string for label on right side)

@cmdcolin cmdcolin removed the in progress currently being worked on label Dec 10, 2015
nathandunn added a commit that referenced this pull request Feb 23, 2016
Add ability to specify "multiple highlights" or bookmarks
@nathandunn nathandunn merged commit 5b0477e into master Feb 23, 2016
@cmdcolin
Copy link
Contributor Author

Let me know if there are any bugs/needed additions for this. I had one more commit that sort of reorganized the code but it might not be necessary. Also @erasche picked this up and added some stuff to the jbrowse configuration guide http://gmod.org/wiki/JBrowse_Configuration_Guide#addBookmarks

@nathandunn
Copy link
Contributor

Most likely any commits I make to it will go away.

What I need is another LocationScaleTrack in GenomeView, but that will come later, . . but not going to do that right now. If you want to re-organize and create another pull request that would be fine.

Nathan

On Feb 23, 2016, at 4:13 PM, Colin Diesh notifications@github.com wrote:

Let me know if there are any bugs/needed additions for this. I had one more commit that sort of reorganized the code but it might not be necessary. Also @erasche https://github.com/erasche picked this up and added some stuff to the jbrowse configuration guide http://gmod.org/wiki/JBrowse_Configuration_Guide#addBookmarks http://gmod.org/wiki/JBrowse_Configuration_Guide#addBookmarks

Reply to this email directly or view it on GitHub #668 (comment).

@cmdcolin cmdcolin deleted the multiple_highlights branch February 29, 2016 21:44
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.

3 participants