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 catalog overview #1865

Merged
merged 7 commits into from
Nov 29, 2023
Merged

feat: emui catalog overview #1865

merged 7 commits into from
Nov 29, 2023

Conversation

Dartoxian
Copy link
Contributor

Description:

This PR implements the first pass of the emui catalog. Additionally:

  • It has a minor tweak to log viewer button colours
  • Refactors the KurtosisBreadcrumbs to no longer be dependent on being on an enclaves route
  • Fixes the application max width to be 1440px, not the 1320px I had incorrectly set it to.

Demo

catalog.mp4

Is this change user facing?

YES

References (if applicable):

  • Figma

@Dartoxian Dartoxian requested a review from adschwartz November 28, 2023 17:59
@adschwartz adschwartz added this pull request to the merge queue Nov 29, 2023
Merged via the queue into main with commit 2f118d9 Nov 29, 2023
35 checks passed
@adschwartz adschwartz deleted the bgazzard/catalog branch November 29, 2023 14:34
github-merge-queue bot pushed a commit that referenced this pull request Nov 29, 2023
🤖 I have created a release *beep* *boop*
---


##
[0.85.37](0.85.36...0.85.37)
(2023-11-29)


### Features

* emui catalog overview
([#1865](#1865))
([2f118d9](2f118d9))

---
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 <[email protected]>
github-merge-queue bot pushed a commit that referenced this pull request Nov 30, 2023
## 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


https://github.com/kurtosis-tech/kurtosis/assets/4419574/19826393-10b2-40de-832b-6970f305a2ce

## Is this change user facing?
YES

## References (if applicable):
* Figma
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.

2 participants