Allow specifying styles for images

[12people]

I would love for Wger to become the central database that all FOSS projects pull workout information from. However, workout apps tend to use very different art styles — wger itself uses line art, openWorkout uses 3D animation, Feeel uses low-poly images.

It'd be great if volunteers could upload graphics in any of these styles and annotate their uploads so that developers could choose which style of illustration to display in their apps.

To achieve this, here's my rough idea of two attributes that could be added for the exerciseimage type:

Style — an enum that could have these values:

• line art
• 3D
• low-poly
• photo
• other

Animated — a boolean specifying whether the image is static or animated (somewhat related to #289)

As for the style attribute, the idea is that more styles could be potentially added depending on the needs of the projects using wger. Aside from the styles used by the FOSS projects listed above, I also propose adding a "photo" style, as photos are arguably the easiest for volunteers to submit and can serve as a starting point for both line art and especially low-poly illustrations.

[rolandgeider]

Hi! This is actually a nice idea that would not require many changes in the DB. I should have code somewhere to allow for users to submit images, must check on that.

[vinayassrao]

I would like to work on this issue.Can anyone guide me...

[rolandgeider]

Sure! We need a new field "style" on ExerciseImage. Since we only allow only the values above, we need to pass them to the choices attribute (take a look e.g. at the gender field in the user profile). We also need to expose the value to the API, but that's just adding the new field to the list in ExerciseImageSerializer

[LucasSD]

Sure! We need a new field "style" on ExerciseImage. Since we only allow only the values above, we need to pass them to the choices attribute (take a look e.g. at the gender field in the user profile). We also need to expose the value to the API, but that's just adding the new field to the list in ExerciseImageSerializer

Hi @rolandgeider and @12people . I've configured my dev environment and it's all working and 1211 tests pass and one is skipped. I can get this issue solved and add tests. Please can you assign it to me? Thanks :)

[rolandgeider]

All yours!

[LucasSD]

All yours!

Thank you!

[LucasSD]

Hi @rolandgeider. Please can you let me know your preferred blank, null and default field options for the new style field?

[rolandgeider]

The current pictures are only line art, so that should be the default in the migrations. As for new submitted ones, I would say "photo"

[LucasSD]

Cool, I've opened a PR :)