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

Suggestion: Invert OSIPI logo text to white #54

Closed
MartinKjunior opened this issue Oct 27, 2023 · 6 comments · Fixed by #66
Closed

Suggestion: Invert OSIPI logo text to white #54

MartinKjunior opened this issue Oct 27, 2023 · 6 comments · Fixed by #66
Assignees

Comments

@MartinKjunior
Copy link
Contributor

MartinKjunior commented Oct 27, 2023

The top bar is dark blue and OSIPI_logo.png text is black which doesn't stand out well. Consider changing text colour to something lighter (e.g. white) to make it legible.
image

@Aditish51
Copy link
Contributor

Hello, This is Aditi. Can you assign this issue to me?

@MartinKjunior
Copy link
Contributor Author

@MRdep I've added this issue because I thought the logo being white would be more visible (ties in with #62 ). If it's okay to change the black text into white, please give Aditi the assignment.

@MRdep
Copy link
Contributor

MRdep commented Mar 8, 2024

Hello, This is Aditi. Can you assign this issue to me?

Hi Aditi, thanks for contributing. Feel free to tackle any of the other open issues.

@Aditish51
Copy link
Contributor

Aditish51 commented Mar 8, 2024

Hi @MRdep Thanks for assigning me this issue.I think it is looking better now.
Screenshot from 2024-03-08 18-26-34

@MartinKjunior
Copy link
Contributor Author

@Aditish51 Could you please check if the colour of the new text is purely white?

@Aditish51
Copy link
Contributor

Hey @MartinKjunior here is the image .I think since it is small maybe that is why it looks faded else if you want I can correct it once again.
OSIPI_logo

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 a pull request may close this issue.

3 participants