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

Changing cog icon to a magnifying glass #310

Merged
merged 1 commit into from
Apr 4, 2017
Merged

Changing cog icon to a magnifying glass #310

merged 1 commit into from
Apr 4, 2017

Conversation

horus68
Copy link
Contributor

@horus68 horus68 commented Dec 14, 2016

Changing cog icon (in the header) to a magnifying glass / search
newcogicon

Why? Magnifying glass its usually considered the search button when the Cog is considered a "configuration tool".
With actual COPS theme people always ask me "where is the search button"?
For me this is now solved on my installs!

Changing cog icon (in the header) to a magnifying glass / search

Why? Magnifying glass its usually considered the search button when the Cog is considered a "configuration tool".
With actual COPS theme people always ask me "where is the search button"?

Note: To play with Search icon size (and make it smaller) see my initial commits to this PR
@horus68
Copy link
Contributor Author

horus68 commented Dec 14, 2016

Note: To the ones that prefer to play with the search icon size (and make it smaller) you can also to this:

in the file templates/default/header.html find line 6 and change
<div title="{{=it.c.i18n.cogAlt}}" class="hicon hicon64 headright"><i class="icon-search" id="searchImage"></i></div>
to
<div title="{{=it.c.i18n.cogAlt}}" class="hicon hicon32 headright"><i class="icon-search icon-2x" id="searchImage"></i></div>

and then, in the file templates/default/styles/style-base.css add this code (around line 163 its ok for it)

.hicon48{
    font-size: 48px;
    line-height: 48px;
    width: 48px;
    height: 48px;
}

the result will be something like this (just ignore the home icon!)
smallicons

@seblucas
Copy link
Owner

seblucas commented Apr 4, 2017

After a lot of thought ..... merged !

@seblucas seblucas merged commit a727335 into seblucas:master Apr 4, 2017
@horus68 horus68 deleted the patch-1 branch May 2, 2017 15:42
dunxd pushed a commit to dunxd/mikespub-cops that referenced this pull request Sep 10, 2023
Changing cog icon to a magnifying glass
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