amahi / android

Amahi Android App
GNU General Public License v3.0
165 stars 278 forks source link

showing images in grid layout #156

Open ak2711 opened 7 years ago

ak2711 commented 7 years ago

grid layout can be used to display images like this one

12121

thecskc commented 7 years ago

This can be done using a recycler view with a grid layout manager with a span of 4

cpg commented 7 years ago

It's a nice idea. How to represent folders of images may also need some thought.

@ak2711 Please specify if you will be working on that or not.

ak2711 commented 7 years ago

i would definitely like to work on it. folders can also be implemented like the following screenshot_2017-03-26-02-12-39

shashvat-kedia commented 6 years ago

I would like to work on this issue

cpg commented 6 years ago

@sd1998 Please don't. Focus on the other issue you said you would be working on first #255

mohitsvnit commented 6 years ago

I would like to work on this issue.

mohitsvnit commented 6 years ago

In "Welcome to amahi" server shares/demo/photos only contains images. Can it also contains folders?

cpg commented 6 years ago

potentially, yes.

maybe do it in two parts. 1) start by assuming it does not. 2) later you can add that functionality if it makes sense.

mohitsvnit commented 6 years ago

OK. Because if folder exist then I have to create one separate fragment for that. First I will work on it assuming that folder does't exist. Can you tell which image format application is supporting?

cpg commented 6 years ago

I don't remember offhand, but for sure png, jpg and possibly gif

mohitsvnit commented 6 years ago

@cpg and @csoni111 Should I create different fragment say ServerFileImageFragment for images having grid structure or should modify ServerFileFragment so that if images comes then it uses grid structure else use linear structure? Please provide your valuable suggestion

csoni111 commented 6 years ago

I think it would be better to modify ServerFileFragment. Duplicating the whole code for a single line change does not make sense.

mohitsvnit commented 6 years ago

@csoni111 I am thinking to change Listview into RecyclerView. What do you say?

csoni111 commented 6 years ago

Yes do that but make sure you don't break any previous functionality.

mohitsvnit commented 6 years ago

@csoni111 functionality of ServerFileAdapter class is depends on multiple classes. Entire architecture seems really complex. I will try to complete it asap.