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

link to messageHeader.css instead of aboutMessage.css #201

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jobisoft
Copy link
Contributor

@jobisoft jobisoft requested a review from darktrojan September 17, 2024 15:13
@jobisoft
Copy link
Contributor Author

Relevant to issue #199

@jobisoft jobisoft mentioned this pull request Sep 17, 2024
19 tasks
@hellsworth
Copy link
Contributor

yeah it's strange.. I can't find any evidence of an aboutMessage.css even existing within the last 3 years, so I'll need to defer to a front end person that is more knowledgeable about where the about3pane stuff is actually located.

@Alecaddd can you verify that https://searchfox.org/comm-central/source/mail/themes/shared/mail/messageHeader.css is the right link to use in this context?

@hellsworth
Copy link
Contributor

I've reached out to the front end devs to get some guidance here. I think that maybe you're right that messageHeader.css is what we want to link to but I'd like to be sure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants