Page MenuHomePhabricator
Create Task
Maniphest T7876

efl.gfx.image_load_controller
Closed, ResolvedPublic
Actions

Description

interface Efl.Gfx.Image_Load_Controller
├ (P) load_size
├ (P) load_dpi
├ (P) load_region_support
├ (P) load_region
├ (P) load_orientation
├ (P) load_scale_down
├ (P) load_skip_header
├ (M) load_async_start
├ (M) load_async_cancel

Details

Commits
D10125 / rEFLb59c00864c1e: api: mark Efl.Gfx.Image_Load_Controller stable
D10128 / rEFL595987ae1775: efl/image: remove load_controller events

Related Objects
Search...

StatusAssignedTask
 ResolvedNoneT7873 efl.ui.image
 ResolvedNoneT7880 efl.ui.bg
 ResolvedNoneT7876 efl.gfx.image_load_controller
bu5hm4n created this task.May 3 2019, 11:33 AM
bu5hm4n triaged this task as TODO priority.
zmike added a comment.Aug 30 2019, 11:10 AM

@woohyun do you need this now?

zmike moved this task from Backlog to Evaluating on the efl: api board.Sep 16 2019, 7:19 AM

I think this has to be looked at now.

segfaultxavi added a comment.Sep 16 2019, 7:56 AM

API looks strange, because it has load_async methods but no regular load. I understand that comes from Efl.File, but still, feels this feels like "everything we could not stuff into generic File or Image interfaces".
Besides this general comment, I see nothing obviously wrong.
I just updated the docs a bit.

zmike added a comment.Sep 16 2019, 8:09 AM

No, that's exactly what this class is.

zmike added a comment.Sep 24 2019, 1:43 PM

I'm removing the events from this since they duplicate the functionality of efl.gfx.image's events and there's probably no case when one would be used without the other.

zmike moved this task from Evaluating to Stabilized on the efl: api board.Sep 24 2019, 9:53 PM
zmike updated the task description. (Show Details)Sep 25 2019, 7:29 AM