-
Notifications
You must be signed in to change notification settings - Fork 122
#167 #224
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
#167 #224
Conversation
|
@martajurcevic cini mi se da imas konflikt. Morati ces rebasati i ponovno pushati |
beba9e9 to
557e063
Compare
|
@martajurcevic I ovaj container od esktra tagova nije dobar. Ja san mu prominija boju samo da vidiš |
| import {Content, Root, Trigger} from '@radix-ui/react-hover-card'; | ||
| import cx from 'classnames'; | ||
| import {Cross1Icon, PlusIcon} from '@radix-ui/react-icons'; | ||
| import {ChevronDownIcon, Cross1Icon, PlusIcon} from '@radix-ui/react-icons'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Poigraj se da umisto radix-ui ikona koristimo https://heroicons.com/, već ih imamo importane, tako da pronađi njihovu varijantu ovih ikona i koristi njih umisto radix-ui ikona.
| /> | ||
| {showAllTags && ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Prazan red fali
| /> | ||
| {showAllTags && ( | ||
| <div | ||
| className="absolute z-10 mt-2 w-full space-y-1 border-0 bg-white py-1 shadow-lg" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
position:absolute kaže elementu da se ne pridržava očekivanih pravila HTML elemenata. To je 99% razlog zašto ti je širina dropdown containera manja od širine samih tagova.
Pronađi dobro rješenje za to, možda je samo stavit fiksnu širinu, ali triba bit pažljiv s tim, pa testiraj sa tagovima različitih dužina imena.
…ing and a dropdown listing the rest of them
|
@martajurcevic čini mi se da si sad appendala i force-pushala, iako smo rekli da ne želimo to radit kad radiš promjene na reviewanom kodu. |
|
[client] SonarCloud Quality Gate failed. |










No description provided.