search

hoangduit / openmeetings

Automatically exported from code.google.com/p/openmeetings
0 stars 0 forks source link

List of file types on upload is too long #657

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Version 0.7 rc2

Client and server: Mandriva Linux Spring 2008

Client-side usability: When I select a file to upload, the large number of
the "allowed file types" makes the scrollbar difficult to access without
constantly pulling the window left and right.

Possible fix: split into subgroups for text files / image files

Original issue reported on code.google.com by jdavid.e...@gmail.com on 11 Mar 2009 at 11:23

Attachments:

GoogleCodeExporter commented 9 years ago 
Hi!

I made a patch that resolves this problem. I divided the list of file masks in 
the
following categories: Documents, Pictures, and All Allowed. Also, I removed the
filemasks from the titles.

WBR, CTpaHHoe

Original comment by CTpaH...@gmail.com on 12 May 2009 at 8:54

Attachments:

GoogleCodeExporter commented 9 years ago 
Vasya, how your patch makes a scrollbar easier to access? AFAIK, you kept the 
initial
long list.

Why two extension sets you've added don't list extension actual extensions?

Original comment by alexei.f...@gmail.com on 15 May 2009 at 5:10

GoogleCodeExporter commented 9 years ago 
Committed at r2020. Please check that the changed patch solves the problem.

Original comment by alexei.f...@gmail.com on 15 May 2009 at 5:55

GoogleCodeExporter commented 9 years ago 
Yes, this patch solves this issue and introduces a new.
This patch does not contain a mask for all supported file types. This may be 
inconvenient for the user.

Original comment by CTpaH...@gmail.com on 16 May 2009 at 7:07

GoogleCodeExporter commented 9 years ago 
That was my humble attempt to fix your patch so it address the initial problem. 
I
agree, it was not perfect, and you are welcome to submit a better patch.

Original comment by alexei.f...@gmail.com on 18 May 2009 at 9:11

GoogleCodeExporter commented 9 years ago 
Yes, I really made a mistake. I took advantage of the proposed decision, but 
did not 
correct the main thing. 
I also noticed that r2020 has introduced a little bug. Lost keyword "var", as 
did local 
variable global. This is evil, so I send a new better patch as soon as possible.

Original comment by CTpaH...@gmail.com on 18 May 2009 at 9:27

GoogleCodeExporter commented 9 years ago 
Hi. I improved my patch. At this time, I think it solves the problem. 
PS: Also, I attach a screenshot of how it looks now in Debian.

Original comment by CTpaH...@gmail.com on 18 May 2009 at 10:51

Attachments:

GoogleCodeExporter commented 9 years ago 
Fixed with r2035

Original comment by volkov.r...@gmail.com on 26 May 2009 at 1:18

GoogleCodeExporter commented 9 years ago

Original comment by seba.wag...@gmail.com on 29 Aug 2011 at 8:03