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

fix duplicate icons #3846

Merged
merged 32 commits into from
Nov 7, 2023
Merged

fix duplicate icons #3846

merged 32 commits into from
Nov 7, 2023

Conversation

AsmaaNabilBakr
Copy link
Contributor

@AsmaaNabilBakr AsmaaNabilBakr commented Jul 16, 2023

Closes #3782

What changed?
replace policies nav item and audit icons (filled, outlined) with one icon that handled with a prop.

image

Why was this change made?

How was this change implemented?

How did you validate the change?

  • manual test

Release notes

Documentation Changes

@AsmaaNabilBakr AsmaaNabilBakr added the area/ui Issues that require front-end work label Jul 16, 2023
@AsmaaNabilBakr
Copy link
Contributor Author

@joshri stroke-fill class break the audit icon. do you have any suggestion to handle it?

image

@AsmaaNabilBakr AsmaaNabilBakr marked this pull request as ready for review July 24, 2023 07:53
@joshri
Copy link
Contributor

joshri commented Jul 26, 2023

The line element that has the stroke-fill class applied to it needs to be set to white in both dark mode and light mode correct? Have you tried just setting that to white and removing the stroke fill class?

@AsmaaNabilBakr
Copy link
Contributor Author

@joshri @TheGostKasper now it works.
thanks @alfaris @mmoulian

image

yitsushi and others added 11 commits November 6, 2023 10:29
With the update, some other calls had to be updated:
- `NewDiscoveryRESTMapper` expects an extra `HTTPClient` argument
- `client` does not have `NewDelegatingClient` anymore, instead we can
  create the same resource with `client.New(...)`

Resolves #3812

References:
- #3812
- kubernetes-sigs/controller-runtime#2150
- https://github.com/kubernetes-sigs/controller-runtime/releases/tag/v0.15.0

Signed-off-by: Balazs Nadasdi <balazs@weave.works>
…ith updated controller-runtime

Signed-off-by: Balazs Nadasdi <balazs@weave.works>
* cleanup datatable

* sort filtered items by `sortValue || value`
Co-authored-by: Yiannis Triantafyllopoulos <8741709+yiannistri@users.noreply.github.com>
…nd `CheckboxActions` components, with the new Sync/Suspend/Resume controls (the `SyncControl` component) (#4080)

* Create the new `SyncControls` component for Sync/Suspend/Resume controls.

* Move all components, related to syncing and suspending objects (existing `SyncActions` and `CheckboxActions` and new `SyncControls` and `ResumeIcon`), to the `Sync` folder.

* Update the related UI snapshot.

* Add `SyncControls` to exports.

* Move custom actions to the start (left) of `SyncControls` buttons.

* Re-arrange icons in `IconType` alphabetically.
Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.51.0 to 1.56.3.
- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.51.0...v1.56.3)

---
updated-dependencies:
- dependency-name: google.golang.org/grpc
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
@AsmaaNabilBakr AsmaaNabilBakr merged commit 7cc50c4 into main Nov 7, 2023
17 checks passed
@AsmaaNabilBakr AsmaaNabilBakr deleted the update-policy-icon branch November 7, 2023 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ui Issues that require front-end work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove duplicate Policies Icons
7 participants