zhevron / tome Goto Github PK

The content of the View layout is not centered. See the attached image.

We need to implement functionality that allows a player to have more than one profile. Useful for when you want to change appearance between set versions.

Queries should be throttled for all types to conserve bandwidth when hovering several people a lot. Currently, it will send a new query every time you mouseover someone.

Using newlines in description or appearance causes Tome data not to display on other clients.

"Put an indent at the beginning of a paragraph that was already there. First word was No. Placed the indent but moved the cursor to after the N." - Nitzi

A way to view characters contained within the cache, in case a player feels like going back and reading somebody's data if they're not around, as well as providing an interface for clearing the cache so people don't need to constantly ask us how to clear it.

Concept is available alongside the general Settings concept here:
https://docs.google.com/drawings/d/1kisP-mYlWaPY-FoJKC6NUWyBmT96yXbPwlo05CLENE8/edit?usp=sharing

Seems to just be my client, but I am running master, so this could be a potential issue for others. The indent still appears and functions as intended, but it throws up this error regardless.

Error: Incorrect function usage. Parameters: (userdata: ), 405, 406 Parameter types: userdata, number, number Function documentation: Sets the current bounds of the selected text. Call with no arguments to remove the current selection. RiftTextfield:SetSelection() -- void RiftTextfield:SetSelection(begin, end) -- number, number Parameters: begin: The new beginning of the selected text, in the same format SetCursor uses. Must be an integer and smaller than "end". end: The new end of the selected text, in the same format SetCursor uses. Must be an integer and larger than "begin". In Tome / Tome.Tome_UI_Layout_Character_Appearance_Text_Textfield:Event.UI.Input.Key.Up stack traceback: [C]: ? [C]: in function 'SetSelection' Tome/Tome.Widget.TextArea.lua:316: in function <Tome/Tome.Widget.TextArea.lua:299>

Error: Tome/Tome.Widget.Draggable.lua:74: attempt to call method 'GetParent' (a nil value)
In Tome / Tome.Tome_Character_WindowBorder:Event.UI.Input.Mouse.Cursor.Move
stack traceback:
[C]: in function 'GetParent'
Tome/Tome.Widget.Draggable.lua:74: in function <Tome/Tome.Widget.Draggable.lua:72>

Occurs when mousing over the window

Using the Tome button or the /tome show command opens an empty Rift window with no other UI elements.

Calling global functions like string.format and table.remove causes the Lua interpreter to lookup the functions in the global table every time. Copying these functions into Tome's local scope will therefore be a direct performance increase and will lower the required CPU load.

Typing in the Appearance and History fields of the Character layout does not properly activate the Save button. Current workaround is to type something into the Prefix, Name, Suffix, Age, Height or Weight boxes. By the looks of it, the text field in use doesn't respond properly to key up events. I imagine the fix for this will be in the feature/new-widgets branch.

When clicking the close button, the text fields retain key focus causing all keys to keep writing in the text field until escape is used to drop focus.

A lot of testers have been confused about that they need to click the top of the field to enable it and have mistaken it for a bug. (is it a bug?)

Would it be possible for clicking anywhere on the field to enable focus while the field is blank?

Adding a tooltip on mouseover of features to provide a brief explanation of what they are, and what their intended use is. Obvious examples would be to explain Tutor Mode, or displaying that /tome ic and /tome ooc can be used instead of the IC/OOC button.

Users should be able to hide the tooltip during combat since it could clutter the screen and cover important parts of the interface/game world while active.

A version check should be implemented to notify users that they're running an out-of-date version of the addon. Possibly include this in the data messages.

After using /tome clear, reloading the UI is required in order for the information to display again. Mousing over a character and typing /tome show results in an error message stating that the character is not found in the cache.

The addon needs a load message to inform the user it has correctly booted, as well as supplying them with the /tome and /tome help commands.

Clicking on the text fields sometimes causes all text written there to delete (as if the text written was classified as the temporary title text)

This seemed to be caused by clicking in a space of the description box which isn't occupied by text, causing all the text to highlight and the next keypress to overwrite.

The editable text areas only display one line of text at a time, even if multiple lines of text are written.

The text areas used for appearance and history fields does not have word wrapping capabilities. This must be added since it will annoy a whole lot of people.

Error: Tome/Tome.Data.lua:147: attempt to perform arithmetic on field 'Throttle' (a nil value)
In Tome / Tome_Tooltip_Event_Tooltip, event Event.Tooltip
stack traceback:
[C]: in function '__add'
Tome/Tome.Data.lua:147: in function 'Query'
Tome/Tome.Tooltip.lua:113: in function 'Update'
Tome/Tome.Tooltip.lua:266: in function <Tome/Tome.Tooltip.lua:240>

Mousing over anyone causes this error.

When deselecting a target, the Tome button does not hide.

The borders have murdered my eyes and sacrificed them to Cthulhu. We need to tone this stuff down. Seriously.

The appearance dropdown overlaps the experience flag dropdown.

Scrollbar will not move past the initial limit it gets even if the text exceeds it