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

Use list_item for the component list in InstancePath::data_ui #6309

Merged
merged 11 commits into from
Jun 4, 2024

Conversation

abey79
Copy link
Contributor

@abey79 abey79 commented May 13, 2024

What

This PR uses list_item for the component list in the entity path selection panel and tooltip.

Blocked by emilk/egui#4471 to resolve the tooltip "first frame flicker"

TODO:

selection panel

before:

image

after:

image

Note: the hover behaviour is the biggest change here: now full span, previously just on the left-column button

tooltip

before:

image

after:
image

Checklist

  • I have read and agree to Contributor Guide and the Code of Conduct
  • I've included a screenshot or gif (if applicable)
  • I have tested the web demo (if applicable):
  • The PR title and labels are set such as to maximize their usefulness for the next release's CHANGELOG
  • If applicable, add a new check to the release checklist!

To run all checks from main, comment on the PR with @rerun-bot full-check.

@abey79 abey79 added ui concerns graphical user interface include in changelog labels May 13, 2024
Base automatically changed from antoine/data-ui-list-update to main May 14, 2024 11:43
@abey79 abey79 force-pushed the antoine/list-item-component-list branch 2 times, most recently from d994e2e to 06573e2 Compare May 14, 2024 16:27
@abey79 abey79 changed the base branch from main to antoine/propcontent-exact-width May 14, 2024 16:34
@abey79 abey79 added the blocked can't make progress right now label May 14, 2024
@abey79 abey79 marked this pull request as ready for review May 14, 2024 16:37
@Wumpf
Copy link
Member

Wumpf commented May 15, 2024

before screenshots would be nice. my memory is pretty bad with these things :)

@abey79
Copy link
Contributor Author

abey79 commented May 15, 2024

before screenshots would be nice. my memory is pretty bad with these things :)

Done (though visually it doesn't make such a difference on screenshots)

crates/re_data_ui/src/instance_path.rs Outdated Show resolved Hide resolved
Copy link

github-actions bot commented May 20, 2024

Deployed docs

Commit Link
e47d6cd https://landing-bpxphhc9m-rerun.vercel.app/docs

@abey79 abey79 force-pushed the antoine/list-item-component-list branch from 3ee706f to d2e86d1 Compare May 20, 2024 16:44
@abey79 abey79 changed the base branch from antoine/propcontent-exact-width to main May 20, 2024 16:46
@abey79 abey79 changed the title Use list_item2 for the component list in InstancePath::data_ui Use list_item for the component list in InstancePath::data_ui Jun 4, 2024
@abey79 abey79 removed the blocked can't make progress right now label Jun 4, 2024
@abey79 abey79 merged commit aaf9d5e into main Jun 4, 2024
40 checks passed
@abey79 abey79 deleted the antoine/list-item-component-list branch June 4, 2024 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
include in changelog ui concerns graphical user interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use the ListItem style for the component list (in the Selection Panel when an entity is selected)
3 participants