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

Oe property work #1763

Merged
merged 3 commits into from Oct 21, 2022
Merged

Oe property work #1763

merged 3 commits into from Oct 21, 2022

Conversation

hunger
Copy link
Member

@hunger hunger commented Oct 21, 2022

Some pretty stand-alone commits:

  • Remove the type colomn from the table
  • Clicking on an defined property takes you to the property definition
  • Support grouping as supported by the new LSP

@hunger
Copy link
Member Author

hunger commented Oct 21, 2022

Screenshot:

image

tools/online_editor/src/properties_widget.ts Outdated Show resolved Hide resolved
tools/online_editor/styles/content.css Outdated Show resolved Hide resolved
tools/online_editor/src/properties_widget.ts Outdated Show resolved Hide resolved
Merge the type information of the properties into the value column as a
tooltip.

Also add some tag into the output via CSS: This should be replaced with
proper icons!
The LSP now groups attributes. Support that in the online editor
@hunger hunger merged commit a0061bd into slint-ui:master Oct 21, 2022
@hunger hunger deleted the oe_property_work branch October 21, 2022 16:13
@hunger
Copy link
Member Author

hunger commented Oct 21, 2022

Last push was a rebase on top of #1759 only.

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.

None yet

2 participants