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

Polish sharing for tracings and datasets #3029

Merged
merged 10 commits into from
Aug 13, 2018
Merged

Polish sharing for tracings and datasets #3029

merged 10 commits into from
Aug 13, 2018

Conversation

philippotto
Copy link
Member

@philippotto philippotto commented Aug 10, 2018

URL of deployed dev instance (used for testing):

Steps to test:

  • test sharing modal for tracings
  • test sharing UI for dataset settings

Issues:


Screenshots:

Default
image

For not public dataset (as admin):
image

After clicking "here":
image

For read only mode:
image

Dataset editing (not public):
image

Dataset editing (public):
image

Tooltip:
image

@philippotto philippotto self-assigned this Aug 10, 2018
@philippotto philippotto requested a review from daniel-wer August 10, 2018 10:31
@philippotto philippotto changed the title Polish sharing Polish sharing for tracings and datasets Aug 10, 2018
Copy link
Member

@daniel-wer daniel-wer left a comment

Choose a reason for hiding this comment

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

Awesome, much easier to understand and use now, works very well :)

</Button>
</Input.Group>
<Hint style={{ margin: "6px 12px" }}>
This link points directly to the current position, zoom value and tree node. Consider
Copy link
Member

Choose a reason for hiding this comment

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

Maybe better: "This link includes the current position, zoom value and active tree node. ..."

style={{ width: "10%" }}
icon="retweet"
>
Renew
Copy link
Member

Choose a reason for hiding this comment

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

Good renaming :)

@@ -114,19 +114,6 @@ class DatasetView extends React.PureComponent<Props, State> {
}
};

updateDataset = (newDataset: DatasetType) => {
Copy link
Member

Choose a reason for hiding this comment

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

I love me some dead code removal :)

@philippotto philippotto merged commit 50cbed6 into master Aug 13, 2018
@philippotto philippotto deleted the polish-sharing branch September 18, 2018 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Polish Dataset/Annotation Sharing
2 participants