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: emui package details page #1873

Merged
merged 7 commits into from Nov 30, 2023
Merged

Conversation

Dartoxian
Copy link
Contributor

Description:

This PR implements the emui package details page.

The change @adschwartz to refactor keyboard listener code requested in #1865 is also included.

It also fixes the poor rendering of the catalog logos on chrome.

Additionally it implements feedback Tise left on the designs for me to pick up, specifically:

  • Value card icon appearance
  • Use preferred round star icon
  • Update the run button appearance and interaction behaviour to match the designs.

Finally, quite a large addition is included to support the combination of react-markdown and chakra in the KurtosisMarkdown component which includes basic chakra implementations of the html components. This is required because CSSReset (part of chakra theme) unsets all of the styling that would apply to markdown.

Short demo

packageDetails.mp4

Is this change user facing?

YES

References (if applicable):

  • Figma

@adschwartz adschwartz added this pull request to the merge queue Nov 30, 2023
Merged via the queue into main with commit e2b75b2 Nov 30, 2023
35 checks passed
@adschwartz adschwartz deleted the bgazzard/emui-design-feedback branch November 30, 2023 22:33
github-merge-queue bot pushed a commit that referenced this pull request Dec 1, 2023
🤖 I have created a release *beep* *boop*
---


##
[0.85.39](0.85.38...0.85.39)
(2023-11-30)


### Features

* emui package details page
([#1873](#1873))
([e2b75b2](e2b75b2))
* User service ports Traefik Docker labels
([#1871](#1871))
([d18f20e](d18f20e))


### Bug Fixes

* move log collector creation logic
([#1870](#1870))
([b695e27](b695e27))
* service name collision error message
([#1863](#1863))
([164b316](164b316))
* Update custom Nix dev deps to work on also linux
([#1862](#1862))
([d11cd37](d11cd37))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: kurtosisbot <kurtosisbot@users.noreply.github.com>
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