search

jongough / ocpn_draw_pi

OpenCPN general drawing plug in
7 stars 17 forks source link

move main icon to beginning - don't merge, just read please #520

Closed rgleason closed 1 year ago

rgleason commented 1 year ago

Jon, We need to move the main icon up earlier. Ocpn_draw as currently configured is bombing out OpenCPN! Dave identified this for us his explanation is better.

Message for Jon: The main plugin icon must be available always, even before the Init() call. This is where we get the icon for the PIM list. So I moved the icon build back to the plugin CTOR.

Thanks.

Line 384 // Create the PlugIn icons m_pODicons = new ODicons();

Line 547 // m_pODicons = new ODicons();

jongough commented 1 year ago

New pre-release in beta, v1.8.21.1, with the requested change.

rgleason commented 1 year ago

Thanks