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

Retina/HiRes icon support #11

Closed
LukasReschke opened this issue Oct 23, 2012 · 26 comments
Closed

Retina/HiRes icon support #11

LukasReschke opened this issue Oct 23, 2012 · 26 comments

Comments

@LukasReschke
Copy link
Member

Currently the ownCloud icon looks pixelated under OS X with a retina MBP.
It might be a good idea to have a retina ready icon in the menu bar and the about section.

@ghost ghost assigned dragotin Oct 23, 2012
@danimo
Copy link
Contributor

danimo commented Oct 23, 2012

This needs Qt support from upstream.

@ghost ghost assigned danimo Oct 23, 2012
@danimo
Copy link
Contributor

danimo commented Oct 23, 2012

https://codereview.qt-project.org/#change,33266 is the change we need to track.

@jancborchardt
Copy link
Member

@danimo the upstream issue you referred is marked as »Abandoned«. But the same author made a new one which is merged: https://codereview.qt-project.org/#change,49767

Does that set the base for using retina icons?

@danimo
Copy link
Contributor

danimo commented Apr 7, 2014

@jancborchardt I'd need the following icons in hidpi (@2x), so basically twice the size:

resources/activity.png
resources/dialog-cancel.png
resources/dialog-close.png
resources/dialog-ok.png
resources/folder-grey.png
resources/folder-remote-32.png
resources/folder-remote.png
resources/folder-sync.png
resources/lock-http.png
resources/lock-https.png
resources/network.png
resources/owncloud_logo_blue.png
resources/settings.png
resources/task-ongoing.png
resources/view-refresh.png
resources/warning-16.png

@ladrua
Copy link

ladrua commented Sep 10, 2014

So I can just add state-ok-32@2x.png file to my theme icons? so retina would be 64x64?

@LukasReschke
Copy link
Member Author

@jancborchardt Do you have those icons in double size somewhere? - It's somewhat annoying to have everything in ownCloud Retina ready except our native application ;-)

@LukasReschke
Copy link
Member Author

screen shot 2014-10-09 at 14 21 12

@jancborchardt
Copy link
Member

Right – I just mailed Achim about it. He made the icons for the desktop client.

@jancborchardt
Copy link
Member

@danimo @dragotin did you see Achim’s reply with the icons?

@mcmanimie
Copy link

Will the retina icons be included in the next release?

@danimo
Copy link
Contributor

danimo commented Feb 12, 2015

@LukasReschke
Copy link
Member Author

🚀

@danimo
Copy link
Contributor

danimo commented Feb 12, 2015

Try now :)

@LukasReschke
Copy link
Member Author

🚀 Looks totally awesome 🚀

Thanks!!!

@danimo danimo added the ReadyToTest QA, please validate the fix/enhancement label Feb 12, 2015
@danimo
Copy link
Contributor

danimo commented Feb 12, 2015

@lmaestro Do you have a retina mac avail?

@jancborchardt
Copy link
Member

@danimo the proper retina icons for the settings dialog and for the lock icon are now also available – check the desktop-homies share. ;) (Please add the SVG sources in the repo! All files are compressed.)

@dragotin
Copy link
Contributor

@LukasReschke screenshot - or it does not exist ;-)

@LukasReschke
Copy link
Member Author

screen shot 2015-02-12 at 21 14 59

@pulse00
Copy link

pulse00 commented Feb 13, 2015

looks great, thanks!

@jancborchardt
Copy link
Member

It looks great and I think the activity icon is very fitting. If anything, the icons for »General« and »Network« look a bit out of place. (Pale color on Network in comparison to Account.)

@danimo
Copy link
Contributor

danimo commented Feb 16, 2015

@jancborchardt Talk to Cuppertino. Those are their recommended icons :)

@jancborchardt
Copy link
Member

Haha, ok. So I would say we close the case cause the new icons I gave you are retina and look better than the KDE-like icon set? ;)

@luciamaestro luciamaestro removed the ReadyToTest QA, please validate the fix/enhancement label Feb 19, 2015
ogoffart added a commit that referenced this issue Jan 7, 2016
The Qt HTTP thread calls authenticationRequired (indirectly) using a
BlockingQueuedConnection. So when we call invalidateToken from slot
connected to this signal and end up calling QNAM::clearAccessCache which
waits on the thread for 5 seconds

Backtraces:

Qt HTTP thread:
 #0  0x00007ffff20c707f in pthread_cond_wait@@GLIBC_2.3.2 ()
 #1  0x00007ffff43f0c0b in QWaitConditionPrivate::wait
 #2  QWaitCondition::wait
 #3  0x00007ffff43ea06b in QSemaphore::acquire
 #4  0x00007ffff45dcf6f in QMetaObject::activate
[...]
 #9  0x00007ffff45dd607 in QMetaObject::activate
 #10 0x00007ffff4edbaf7 in QHttpNetworkReply::authenticationRequired
 #11 0x00007ffff4e0b2b4 in QHttpNetworkConnectionPrivate::handleAuthenticateChallenge
 #12 0x00007ffff4e10753 in QHttpNetworkConnectionChannel::handleStatus
 #13 0x00007ffff4e11cc9 in QHttpNetworkConnectionChannel::allDone
 #14 0x00007ffff4e14605 in QHttpProtocolHandler::_q_receiveReply

Main Thread:
 #0  0x00007ffff20c7428 in pthread_cond_timedwait@@GLIBC_2.3.2 ()
 #1  0x00007ffff43f0b56 in QWaitConditionPrivate::wait_relative (time=5000, this=0x136c580)
 #2  QWaitConditionPrivate::wait (time=5000, this=0x136c580)
 #3  QWaitCondition::wait (this=this@entry=0x136c788, mutex=mutex@entry=0x136c760, time=time@entry=5000)
 #4  0x00007ffff43efa6e in QThread::wait (this=<optimized out>, time=time@entry=5000)
 #5  0x00007ffff4e1edd3 in QNetworkAccessManagerPrivate::clearCache
 #6  0x00007ffff7b6fb03 in OCC::HttpCredentials::invalidateToken()
 #7  0x000000000057adb4 in OCC::AccountState::slotInvalidCredentials()
 #8  0x000000000057ac76 in OCC::AccountState::slotConnectionValidatorResult(OCC::ConnectionValidator::Status, QStringList const&) ()
 #9  0x00000000005ab45c in OCC::AccountState::qt_static_metacall(QObject*, QMetaObject::Call, int, void**)
 #10 0x00007ffff45dcd30 in QMetaObject::activate
 #11 0x00007ffff7b78671 in OCC::ConnectionValidator::connectionResult(OCC::ConnectionValidator::Status, QStringList) ()
 #12 0x00007ffff7ae2514 in OCC::ConnectionValidator::reportResult(OCC::ConnectionValidator::Status) ()
 #13 0x00007ffff7ae39b7 in OCC::ConnectionValidator::slotAuthFailed(QNetworkReply*) ()
 #14 0x00007ffff7b784a9 in OCC::ConnectionValidator::qt_static_metacall(QObject*, QMetaObject::Call, int, void**) ()
 #15 0x00007ffff45dcd30 in QMetaObject::activate
 #16 0x00007ffff7b766dc in OCC::AbstractNetworkJob::networkError(QNetworkReply*)
 #17 0x00007ffff7af9f6e in OCC::AbstractNetworkJob::slotFinished()
 #18 0x00007ffff7b7654d in OCC::AbstractNetworkJob::qt_static_metacall(QObject*, QMetaObject::Call, int, void**) ()
 #20 0x00007ffff45dd607 in QMetaObject::activate
 #21 0x00007ffff4edd143 in QNetworkReply::finished
 #22 0x00007ffff4e3fec7 in QNetworkReplyHttpImplPrivate::finished
 #23 0x00007ffff4e41818 in QNetworkReplyHttpImpl::close
 #24 0x00007ffff7b7047b in OCC::HttpCredentials::slotAuthentication(QNetworkReply*, QAuthenticator*) ()
 #25 0x00007ffff7b79092 in OCC::HttpCredentials::qt_static_metacall(QObject*, QMetaObject::Call, int, void**) ()
 #27 0x00007ffff45dd607 in QMetaObject::activate
 #28 0x00007ffff4e1d6fb in QNetworkAccessManager::authenticationRequired
 #29 0x00007ffff4e1ea07 in QNetworkAccessManagerPrivate::authenticationRequired
 #30 0x00007ffff4e3c784 in QNetworkReplyHttpImplPrivate::httpAuthenticationRequired

Another case of Main Thread:
 #5  0x00007ffff4e1edd3 in QNetworkAccessManagerPrivate::clearCache
 #6  0x00007ffff7b6fb03 in OCC::HttpCredentials::invalidateToken()
 #7  0x000000000057b1e4 in OCC::AccountState::slotInvalidCredentials() ()
 #8  0x00000000005abb8a in OCC::AccountState::qt_static_metacall(QObject*, QMetaObject::Call, int, void**) ()
 #9  0x00007ffff45dcd30 in QMetaObject::activate
 #10 0x00007ffff7b76ed5 in OCC::Account::invalidCredentials() ()
 #11 0x00007ffff7ad55f5 in OCC::Account::handleInvalidCredentials()
 #12 0x00007ffff7afa69a in OCC::AbstractNetworkJob::slotFinished()
 #13 0x00007ffff7b7654d in OCC::AbstractNetworkJob::qt_static_metacall(QObject*, QMetaObject::Call, int, void**) ()
 #15 0x00007ffff45dd607 in QMetaObject::activate
 #16 0x00007ffff4edd143 in QNetworkReply::finished
 #17 0x00007ffff4e3fec7 in QNetworkReplyHttpImplPrivate::finished
 #18 0x00007ffff4e41818 in QNetworkReplyHttpImpl::close
 #19 0x00007ffff7b7047b in OCC::HttpCredentials::slotAuthentication(QNetworkReply*, QAuthenticator*) ()
 #20 0x00007ffff7b79092 in OCC::HttpCredentials::qt_static_metacall(QObject*, QMetaObject::Call, int, void**) ()
 #22 0x00007ffff45dd607 in QMetaObject::activate
 #23 0x00007ffff4e1d6fb in QNetworkAccessManager::authenticationRequired
juliusknorr pushed a commit to juliusknorr/client that referenced this issue Aug 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants