-
Notifications
You must be signed in to change notification settings - Fork 11
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
Added a dataset header component to link to datasets #9
Conversation
src/components/FloatingFlow.vue
Outdated
@@ -168,16 +168,34 @@ export default { | |||
mainTabName: "Flatmap", | |||
zIndex: 1, | |||
showDialogIcons: false, | |||
dockedArray: [{title: "Flatmap", id:1}], | |||
dockedArray: [{title: "Flatmap", id:1}, {title: "Heatmap", id:2}, {title: "Plot", id:3}], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do not think we should have all three dialog docked right at the beginning. It will destroy the flow of the current use case. This can be added in the fetchContent method in the MapPopover module instead.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are number of changes required. Please take a look.
src/components/FloatingFlow.vue
Outdated
entries: [ | ||
{ | ||
resource: "Human", availableSpecies : {"Human":{taxo: "NCBITaxon:9606", | ||
iconClass:"icon-mapicon_human"}, "Rat":{taxo: "NCBITaxon:10114", iconClass:"icon-mapicon_rat"} }, | ||
type: "Flatmap", zIndex:1, mode: "main", | ||
id: 1 | ||
} | ||
id: 1, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See above.
src/components/FloatingDialog.vue
Outdated
<el-main class="dialog-main" :style="mainStyle"> | ||
<DatasetHeader v-if="entry.datasetTitle" class="dataset-header" :title="entry.datasetTitle" :url="entry.datasetUrl"></DatasetHeader> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have to take the height of the header into account. There is now overflow in the dalog. Create another div for the vuers with appropriate size. height: calc(100%-25px)?
- Switch to using an entry object as prop for DatasetHeader - Positions pop up cards based off click location - Switch to copying full entry object in 'FloatingFlow'
src/components/FloatingFlow.vue
Outdated
newEntry.id = ++this.currentCount; | ||
newEntry.zIndex = ++this.zIndex; | ||
this.entries.push(newEntry); | ||
this.onFlowChange('createNewEntry'); | ||
window.newEntry = newEntry |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You missed this one.
src/components/MapPopover.vue
Outdated
@@ -102,6 +102,7 @@ export default { | |||
]; | |||
break; | |||
case "ICN": | |||
console.log('in ICN') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this one too. Thanks
Fix Search for dataset on upstream downstream not working.
No description provided.