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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

opacity on viewing image attachment? #302

Closed
mpaolo opened this issue Nov 8, 2018 · 4 comments
Closed

opacity on viewing image attachment? #302

mpaolo opened this issue Nov 8, 2018 · 4 comments
Assignees
Labels
bug something is wrong in Cypht themes themes module set related

Comments

@mpaolo
Copy link

mpaolo commented Nov 8, 2018

馃悰 Bugreport

Hi,
seems that all but the default theme have
img { opacity: .3 !important; }
for the img tag, hence an attached img displays like behind a semiopaque glass.
Don't see the reason for that, nor an obvious config-dependent option selected,
thus I deem it a bug

thx

Version & Environment

Cypht latest, Debian stretch, nginx+dovecotimapd, vrom Ffox latest on W10pro

Steps to reproduce

just try to view a sen/received img attachment in non-default theme e.g. *blue

@jasonmunro jasonmunro self-assigned this Nov 8, 2018
@jasonmunro jasonmunro added bug something is wrong in Cypht themes themes module set related labels Nov 8, 2018
@jasonmunro
Copy link
Member

Yep, good call. The opacity change is to make the icons blend in a bit better with themes. Just added an override for viewing message image attachments. Overall themes could use some love :)

@mpaolo
Copy link
Author

mpaolo commented Nov 9, 2018

hm I grabbed the latest, seen the changes, then replaced the new .css ... but see no effect. Thing is, seems the browser still sees the previous .css. Purged browser cache, restarted nginx, close/open browser ... to no avail. Wonder how to convince FFox to drop the cached things, or where the heck it's cached :/
Even in another browser wich never visited the site the .css seems the old one - missing the
.msg_text_inner img, .apod_image img { opacity: 1 !important; }
So either it's the nginx or ... am I supposed to re-run the config-gen script?

@jasonmunro
Copy link
Member

Sorry, should have mentioned that you do need to rerun the config gen script. Part of what that does is copy assets from enabled module sets into the site directory, like the theme css files.

@mpaolo
Copy link
Author

mpaolo commented Nov 9, 2018

aha! that did the trick indeed - looks fine now
thanx!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug something is wrong in Cypht themes themes module set related
Projects
None yet
Development

No branches or pull requests

2 participants