Create events add image title in graphic page#2114
Conversation
| </div> | ||
| </div> | ||
| ); | ||
| return <ImageTile image={image} className={className} />; |
There was a problem hiding this comment.
Fint! Men kan du gjøre stylingen litt mindre "forstyrrende" til bildet som ligger bak? Enten alt eller noe av dette kan prøves:
- Gjøre teksten litt mindre
- Gjøre overlayet enda mer gjennomsiktig (men fortsatt lesbart med god kontrast)
- Kanskje legge tekst under selve bildet
ImagePicker componenten sin resizing css osv bør ses på |
There was a problem hiding this comment.
AdminImage trenger vel ikke lenger å være sin egen komponent når den ikke legger til noe lenger, bare bruk ImageTile direkte ImageAdminPage 👍
| box-shadow: 0 0 25px 5px rgba(143, 181, 238, 0.625); | ||
| border: 3px solid $blue; | ||
| transform: scale(1); | ||
| } |
There was a problem hiding this comment.
Lag denne default selected mer clean!
There was a problem hiding this comment.
Gjør alt av selected css shit her ..? Så trenger man ikke ha masse css props osv overalt .. ?
There was a problem hiding this comment.
Ser litt halv jalla ut
| } | ||
| .image_box_selected { | ||
| border: none; | ||
| } |
There was a problem hiding this comment.
Bør nok fjernes å heller gjøre alt av selected state i ImageTile .. ?
| styles.imageContainer, | ||
| className, | ||
| selected && styles.selected, | ||
| selected && selectedClassName, |
| image: ImageDto; | ||
| className?: string; | ||
| selected?: boolean; | ||
| selectedClassName?: string; |
| image={image} | ||
| className={styles.image_box} | ||
| selected={isSelected} | ||
| selectedClassName={styles.image_box_selected} |
robines
changed the title
3 participants
The two pages now inherit structure and styling from the same component, providing a more unified look. This includes the labels.