anna/XivCommon
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

refactor: move InternalAction to native items only

Browse Source
This commit is contained in:
Anna 2021-04-29 16:08:16 -04:00
parent dc593f5751
commit dc596aee66
Signed by: anna
GPG Key ID: 0B391D8F06FCD9E0
1 changed files with 6 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
XivCommon/Functions/ContextMenu.cs
View File

@ -6,7 +6,6 @@ using System.Text;
using Dalamud;
using Dalamud.Game;
using Dalamud.Hooking;
using Dalamud.Plugin;
using FFXIVClientStructs.FFXIV.Component.GUI;
using ValueType = FFXIVClientStructs.FFXIV.Component.GUI.ValueType;
@ -236,7 +235,7 @@ namespace XivCommon.Functions {
// set up the agent to take the appropriate action for this item
*(menuActions + 7 + i) = item switch {
NativeContextMenuItem => item.InternalAction,
NativeContextMenuItem nativeItem => nativeItem.InternalAction,
_ => NoopContextId,
};
@ -311,11 +310,6 @@ namespace XivCommon.Functions {
/// A base context menu item
/// </summary>
public abstract class BaseContextMenuItem {
/// <summary>
/// The action code to be used in the context menu agent for this item.
/// </summary>
public byte InternalAction { get; internal set; }
/// <summary>
/// If this item should be enabled in the menu.
/// </summary>
@ -326,6 +320,11 @@ namespace XivCommon.Functions {
/// A native context menu item
/// </summary>
public sealed class NativeContextMenuItem : BaseContextMenuItem {
/// <summary>
/// The action code to be used in the context menu agent for this item.
/// </summary>
public byte InternalAction { get; }
/// <summary>
/// The name of the context item.
/// </summary>