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.

Sign up for GitHub

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

Jump to bottom

LC favicon - chromium based browser #247

Closed
toskrip opened this issue Jul 26, 2021 · 2 comments · Fixed by #336
Closed

LC favicon - chromium based browser #247

toskrip opened this issue Jul 26, 2021 · 2 comments · Fixed by #336
Assignees
@toskrip
Labels
maintenance Dependency management and legacy code cleanups
Milestone
LibreClinica-1.2

Comments

@toskrip
Copy link
Collaborator

toskrip commented Jul 26, 2021 •
edited
Loading

Description:
Favicon is not displayed on LC instance web interface when accessed via chromium based browser.

Requirements:
Out of the box LC installation.

Steps to follow:

  1. Access LC page

Expected result:
Favicon should be displayed for LC page

Actual result:
No favicon
@toskrip toskrip added the enhancement New feature or request label Jul 26, 2021
@toskrip toskrip added this to the LibreClinica-1.1 milestone Jul 26, 2021
@toskrip toskrip self-assigned this Jul 26, 2021
@toskrip toskrip changed the title LC favicon LC favicon - chromium based browser Jul 26, 2021
@toskrip
Copy link
Collaborator Author

toskrip commented Jul 26, 2021

could be caused by the fact that favicon is not linked in header but is present in html body

@toskrip toskrip removed this from the LibreClinica-1.1 milestone Oct 26, 2021
@toskrip toskrip added this to the LibreClinica-1.2 milestone May 24, 2022
@toskrip
Copy link
Collaborator Author

toskrip commented May 24, 2022

this means remove favicon from footer and identify all headers where it has to be added

@toskrip toskrip added maintenance Dependency management and legacy code cleanups and removed enhancement New feature or request labels May 24, 2022
@toskrip toskrip linked a pull request May 25, 2022 that will close this issue
Various GUI related fixes #336
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@toskrip toskrip

Labels
maintenance Dependency management and legacy code cleanups
Projects
None yet
Milestone
LibreClinica-1.2
Development

Successfully merging a pull request may close this issue.

Various GUI related fixes reliatec-gmbh/LibreClinica
1 participant
@toskrip