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

Force correct font to make magnifying glass appear #5017

Merged
merged 1 commit into from
Nov 11, 2020

Conversation

IsaacPD
Copy link
Contributor

@IsaacPD IsaacPD commented Nov 11, 2020

Fixes: #4903

Description

Force use of FontAwesome with '!important' tag. Effectively this changes the font from "Font Awesome 5 Free" to "FontAwesome" which for some unknown reasons fixes the issue above.

User facing changes
The magnifying glass now appears in the search bars across the skaffold site
Screen Shot 2020-11-10 at 7 20 31 PM

@IsaacPD IsaacPD added the docs-modifications runs the docs preview service on the given PR label Nov 11, 2020
@IsaacPD IsaacPD requested review from a team and tstromberg and removed request for a team November 11, 2020 00:22
@google-cla google-cla bot added the cla: yes label Nov 11, 2020
@IsaacPD IsaacPD requested review from nkubala, tejal29 and a team and removed request for tstromberg November 11, 2020 00:22
@container-tools-bot
Copy link

Please visit http://35.236.77.39:1313 to view changes to the docs.

@container-tools-bot container-tools-bot removed the docs-modifications runs the docs preview service on the given PR label Nov 11, 2020
@container-tools-bot
Copy link

Error creating deployment, please see controller logs for details.

1 similar comment
@container-tools-bot
Copy link

Error creating deployment, please see controller logs for details.

@codecov
Copy link

codecov bot commented Nov 11, 2020

Codecov Report

Merging #5017 (b8d71dc) into master (3a73d83) will increase coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5017      +/-   ##
==========================================
+ Coverage   72.11%   72.15%   +0.04%     
==========================================
  Files         365      365              
  Lines       12773    12807      +34     
==========================================
+ Hits         9211     9241      +30     
- Misses       2877     2879       +2     
- Partials      685      687       +2     
Impacted Files Coverage Δ
pkg/skaffold/docker/parse.go 84.06% <0.00%> (-1.41%) ⬇️
pkg/skaffold/errors/errors.go 97.36% <0.00%> (+0.49%) ⬆️
pkg/skaffold/docker/dependencies.go 75.00% <0.00%> (+2.69%) ⬆️
pkg/skaffold/errors/err_map.go 90.56% <0.00%> (+3.38%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3a73d83...b8d71dc. Read the comment docs.

@IsaacPD IsaacPD added the docs-modifications runs the docs preview service on the given PR label Nov 11, 2020
@container-tools-bot
Copy link

Please visit http://35.236.77.39:1313 to view changes to the docs.

@container-tools-bot container-tools-bot removed the docs-modifications runs the docs preview service on the given PR label Nov 11, 2020
@tejal29
Copy link
Contributor

tejal29 commented Nov 11, 2020

great! its fixed on linux!
image

@tejal29 tejal29 merged commit 8caebb9 into GoogleContainerTools:master Nov 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Documentation: Font Awesome search icon placeholder broken
3 participants