Ticket #267 (closed enhancement: duplicate)

Opened 4 years ago

Last modified 4 years ago

Explicitly inherited method to (re)construct activity GUI after loading(possible) journal entity in sugar-toolkit

Reported by: alsroot Owned by: marcopg
Priority: Low Milestone: 0.86
Component: sugar Version: 0.83.x
Severity: Minor Keywords: tomeu
Cc: Distribution/OS: Unspecified
Bug Status: New

Description

now, read_file() calling placed in map event handler

if I want to construct activity's gui according to loaded jobject
I have to use connect_after("map") and rely on the fact that read_file() will be called before (of course I could call this construction from overloaded read_file but in that case I should count situation w/o loading jobject)

better way is declaring method in Activity class to handle this interval: after init(), read_file() invoking(possible) and before showing gui

Change History

follow-up: ↓ 2   Changed 4 years ago by tomeu

  • keywords tomeu added

Why don't you update your UI in read_file()?

in reply to: ↑ 1 ; follow-up: ↓ 3   Changed 4 years ago by alsroot

Replying to tomeu:

Why don't you update your UI in read_file()?

sometime there is differences between "just construct" and "reconstruct existed"

in reply to: ↑ 2 ; follow-up: ↓ 4   Changed 4 years ago by alsroot

Replying to alsroot:

Replying to tomeu:

Why don't you update your UI in read_file()?

sometime there is differences between "just construct" and "reconstruct existed"

moreover in case of having new method there is only one place to collect logic (otherwise in init() and in read_file())

in reply to: ↑ 3   Changed 4 years ago by tomeu

Replying to alsroot:

Replying to alsroot:

Replying to tomeu:

Why don't you update your UI in read_file()?

sometime there is differences between "just construct" and "reconstruct existed"

moreover in case of having new method there is only one place to collect logic (otherwise in init() and in read_file())

I'm a bit concerned about making the Activity class more complicated. It's already quite messy and that is confusing activity authors.

Cannot we say that activity authors that find themselves in this situation can construct their UI by calling a single function from init and read_file?

  Changed 4 years ago by tomeu

  • milestone changed from Unspecified by Release Team to 0.86

As any solution to this issue would involve API modifications, I'm moving it to 0.86.

  Changed 4 years ago by marcopg

  • priority changed from Unspecified by Maintainer to Normal
  • version changed from Unspecified to 0.83.x
  • status_field changed from Unconfimed to New

  Changed 4 years ago by alsroot

  • priority changed from Normal to Low

  Changed 4 years ago by alsroot

  • status changed from new to closed
  • resolution set to duplicate

duplicate of #428

Note: See TracTickets for help on using tickets.