Stop hardcoding certain component colors #157

Closed
opened 2021-12-07 04:16:35 +02:00 by archseer · 2 comments

The top right heading ("[neomutt-devel...") here has a hard-coded dark blue color: https://meli.delivery/images/screenshots/conversations-view-open-thread.webp

Some of the threading UI and contacts UI too. Searching for "Color::Byte" finds quite a few references.

The top right heading ("[neomutt-devel...") here has a hard-coded dark blue color: https://meli.delivery/images/screenshots/conversations-view-open-thread.webp Some of the threading UI and contacts UI too. Searching for "Color::Byte" finds quite a few references.

Hello! Thanks for the issue. Indeed these Color::Byte references were before themes were introduced in the code base. Their presence is a bug and they should all be replaced by theme values.

Hello! Thanks for the issue. Indeed these `Color::Byte` references were before themes were introduced in the code base. Their presence is a bug and they should all be replaced by theme values.
Manos Pitsidianakis added the
bug
easy
labels 2021-12-07 11:31:10 +02:00

There are some hardcoded colors left in src/components/utilities/widgets.rs:1023 but I added a FIXME comment and I guess they are low priority. Will fix though.

There are some hardcoded colors left in [`src/components/utilities/widgets.rs:1023`](https://git.meli.delivery/meli/meli/src/commit/2eb22a290abb3f37bc77c3bc2771edfb60a1c314/src/components/utilities/widgets.rs#L1023) but I added a `FIXME` comment and I guess they are low priority. Will fix though.
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: meli/meli#157
There is no content yet.