[S60] Widget Menu API needs refinement

• Component: New Bugs
• Status: CLOSED FIXED

[Sornalatha Rathnasamy]

TSW Id : ID:  SYAI-73AS6K 

- Preferences - Key/Value pair implementation must be optimized
- Ensure that only unique menu items APPENDED. Throw exception when new MenuItem call contains same id/text
- Setting the callback after appending the menu item fails
- Menu object is singleton. So no need for new menu()
- Change menu item append behaviour to LIFO
- Fix onshow, onhide behaviour
- int value for set preference for key doesnt work

[Sornalatha Rathnasamy]

Created attachment 15231 [details]
menu api and preferences

[Sornalatha Rathnasamy]

Created attachment 15232 [details]
3.2 patch for menu api and preferences enhancement

[zalan]

Comment on attachment 15232 [details]
3.2 patch for menu api and preferences enhancement

widgetExtension.cpp line#860
>if ( newItem && oldItem && newItem->Text().Compare(newItem->Text()) != 0 )
shouldn't it be something like if ( newItem && oldItem && newItem->Text().CompareF(oldItem->Text()) == 0 ) ?

WidgetPreferences.cpp 
- remove tabs, please.
- line# 225, 251: missing CleanupStakc: PushL
- line# 436: use allocL() instead
- linr# 444: iValue = null; is missing and use allocL() instead.

[Sornalatha Rathnasamy]

Created attachment 15245 [details]
3.2 patch for menu api enhancement

[zalan]

Comment on attachment 15245 [details]
3.2 patch for menu api enhancement

r=me

[zalan]

r23814 -> ccb

[Sornalatha Rathnasamy]

Created attachment 15376 [details]
migrate fixes for SYAI-73AS6K  and SYAI-73ARXK to 3.1

[zalan]

Comment on attachment 15376 [details]
migrate fixes for SYAI-73AS6K  and SYAI-73ARXK to 3.1

r=me

[zalan]

r24160 -> 3.1m

[Sornalatha Rathnasamy]

Created attachment 15491 [details]
3.1 patch

[Sornalatha Rathnasamy]

Created attachment 15494 [details]
3.1 patch

[zalan]

Comment on attachment 15494 [details]
3.1 patch

r=me. the change is not needed.

[Bradley Morrison]

Bulk closing of all s60 platform bugs. 

Sorry for the noise!