DiamondLightSource / DiamondLightSource/sci-react-ui

NavLinks, FooterLinks, NavLink and FooterLink should be using `styled`.

Đang mở
#89 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
accepted priority: low
Ngôn ngữ chính
TypeScript
Star
8
Fork
3
Merge trung bình
3 ngày 15 giờ
Pull request đã merge (30 ngày)
5

Mô tả

Currently NavLinks, FooterLinks, NavLink and FooterLink styles can all be removed by passing in `styles={}` , obviously, breaking the way they are displayed.

Other components are using the function `styled()`, e.g.
```
const BarStyled = styled(Bar)(
({ theme }) => ({
backgroundColor: theme.vars.palette.primary.light
})
);
```

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Hướng nghiên cứu

Xác định các component NavLinks, FooterLinks, NavLink và FooterLink rồi so sánh cách định kiểu của chúng với các component khác sử dụng styled(). Xác minh rằng việc truyền styles={} không còn có thể xóa các style cần thiết cho việc hiển thị chúng, sau đó kiểm tra hành vi của các component bị ảnh hưởng.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
react, typescript
Lĩnh vực
frontend
Loại issue
Tái cấu trúc
Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
45/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.