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

[Activity tab] Message too long when Virus was detected #3676

Closed
stonerl opened this issue Aug 24, 2015 · 3 comments
Closed

[Activity tab] Message too long when Virus was detected #3676

stonerl opened this issue Aug 24, 2015 · 3 comments
Assignees
Milestone

Comments

@stonerl
Copy link

stonerl commented Aug 24, 2015

Expected behaviour

The message shown in the Action-tab, when a Virus was detected, is way to long and does not fit in the window.

Actual behaviour

Here is the message I get:

The item is not synced because of previous errors: Error downloading https://owncloud.example.org/remote.php/webdav/Music/test-virus - server replied: Unsupported Media Type (Virus Eicar-Test-Signature is detected in the file. Upload cannot be completed.)

Steps to reproduce

  1. Create a file with the Eicar-Test-Signature.
  2. Place this file in a sync-folder
  3. Wait for the sync.

Client configuration

Client version: 2.0.0-nightly20150824 (build 2664)

Operating system: OSX 10.10.5

OS language: en-US

@guruz
Copy link
Contributor

guruz commented Aug 25, 2015

@ogoffart Can a tree view easily have multi-line text in one column?

@guruz guruz added this to the 2.0.1-next milestone Aug 25, 2015
@ogoffart
Copy link
Contributor

maybe using QTreeView::setWordWrap

@ogoffart ogoffart modified the milestones: 2.1-next, 2.0.2-next Sep 3, 2015
@guruz guruz changed the title Message too long when Virus was detected [Activity tab] Message too long when Virus was detected Sep 3, 2015
@ogoffart
Copy link
Contributor

ogoffart commented Sep 3, 2015

I tried and the message is ellided with a tooltip, and i think it is good engough for now.

Anyway. We will re-design the activity for 2.1
#3732

@guruz guruz modified the milestones: 2.1.1-nextpatch, 2.1-current Nov 4, 2015
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

3 participants