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

feat(ontology): display description for default and existing props (DSP-1154) #475

Merged
merged 8 commits into from Jun 30, 2021

Conversation

kilchenmann
Copy link
Contributor

resolves DSP-1154

@kilchenmann kilchenmann self-assigned this Jun 29, 2021
@kilchenmann kilchenmann added the enhancement New feature or request label Jun 29, 2021
@kilchenmann
Copy link
Contributor Author

The description from the default properties will be displayed as tooltipp:

Screen Shot 2021-06-29 at 14 58 42

@mdelez could you check the english sentences? Thanks

@gautschr
Copy link
Contributor

I suggest for "Text":
"Short text such as" instead of "Small text like"

A question concerning Date/Time: The inclusion of the new GUI element which allows for a proper display of dates with lower precision than one day - is this another PR?

@kilchenmann
Copy link
Contributor Author

kilchenmann commented Jun 29, 2021

I suggest for "Text":
"Short text such as" instead of "Small text like"

I will update as suggested. → done in 6fb9746

A question concerning Date/Time: The inclusion of the new GUI element which allows for a proper display of dates with lower precision than one day - is this another PR?

The new gui element for dates should already be implemented. So yes, how should the description look like?

@@ -218,18 +235,11 @@ export class DefaultProperties {
{
icon: 'palette',
label: 'Color',
description: 'A simple color value',
Copy link
Collaborator

Choose a reason for hiding this comment

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

We should probably clarify that this is a hexadecimal value. Ex: "A hexadecimal color value"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I am not sure if everyone understands the term "hexadecimal". Apart from that, in the GUI we have the color picker only and the hexadecimal value itself is not displayed anymore (s. DSP-1667)

Copy link
Collaborator

Choose a reason for hiding this comment

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

That's fair :)

}
// {
Copy link
Collaborator

Choose a reason for hiding this comment

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

is this code still needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not sure if we have to reactivate it one day!?

@kilchenmann kilchenmann requested a review from mdelez June 30, 2021 08:27
@kilchenmann kilchenmann merged commit 8be7e55 into main Jun 30, 2021
@kilchenmann kilchenmann deleted the wip/dsp-1154-default-props branch June 30, 2021 08:37
@gautschr
Copy link
Contributor

gautschr commented Jul 1, 2021

We need a kind of dd/mm/yyyy BCE/CE icon and for the short description I suggest "dd/mm/yyyy format; for historical dates"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants