twostars

Huh, nice find. I don't think I've tried to drag/drop skills from the tree in years. Makes me wonder how long that would've gone unnoticed otherwise... Not only are they...

CUISkillTreeDlg isn't handling it like other things. Ordinarily we go: ``` case UIMSG_ICON_UP: // ¾ÆÀÌÄÜ ¸Å´ÏÀú À©µµ¿ìµéÀ» µ¹¾Æ ´Ù´Ï¸é¼­ °Ë»ç.. if ( !CGameProcedure::s_pUIMgr->BroadcastIconDropMsg(CN3UIWndBase::m_sSelectedIconInfo.pItemSelect) ) // ¾ÆÀÌÄÜ À§Ä¡ ¿ø·¡´ë·Î.. IconRestore(); break;...

Unintentionally triggering behaviour (from the caller's perspective) in a method indicated as a check only is probably not the way to go there.

I'm assuming the packet being referred to is the one sent by SaveEvent(), because WIZ_QUEST has many different subopcodes (many of which are not for 1.298, but I suspect the...

Need to implement: - [ ] Loading the list of quest states from the server (subopcode 1). - [ ] Updating quest states (subopcode 2). - [ ] Adding checks...

Looks like the effect file isn't being loaded properly. If you went up in the callstack and determined which effect file it was, you could easily reproduce it by loading...

You might look into why the control's not loaded here, because that seems like more of a problem than the fact it's crashing.

If m_pText_Message is nullptr it means the control's not loaded, which means either the UI needs updating or it's being used for a specialisation of a messagebox UI that isn't...

> Should the Q-Button be hidden if the client can't load the UI or the help-table? No, because that case is exceptional. Those cases won't (or shouldn't) happen when the...

You're honestly better off not trying to shoehorn the new system with the old. Frankly, the new system's painful enough to deal with as it is. Having said that, you...