0 users browsing Bug Reports. | 1 guest  
Main » Bug Reports » UIList Issue not quite resolved it totes is » New reply
Alert
You are about to bump an old thread. This is usually a very bad idea. Please think about what you are about to do before you press the Post button.
New reply
Post help

Presentation

[b]…[/b] — bold type
[i]…[/i] — italic
[u]…[/u] — underlined
[s]…[/s] — strikethrough
[code]…[/code] — code block
[spoiler]…[/spoiler] — spoiler block
[spoiler=…]…[/spoiler]
[source]…[/source] — colorcoded block, assuming C#
[source=…]…[/source] — colorcoded block, specific language[which?]

Links

[img]http://…[/img] — insert image
[url]http://…[/url]
[url=http://…]…[/url]
>>… — link to post by ID
[user=##] — link to user's profile by ID

Quotations

[quote]…[/quote] — untitled quote
[quote=…]…[/quote] — "Posted by …"
[quote="…" id="…"]…[/quote] — ""Post by …" with link by post ID

Embeds


Most plain HTML also allowed.
Thread review
Kawa Done. Will push with the next update.
Mega-Mario If I were you, I'd also add some parentheses to that double ternary operator. I don't think it'd change much how the compiler sees it, but it'd make it more readable.

Just a lil' suggestion :)
Kawa Well, fuck. Implemented. Will push when the next work on lootsets is done.
PillowShout Sorry to make another thread about this, but the other one was closed. Feel free to delete this one after it's been resolved.

Your fix for the UIList Index doesn't seem to work, as trying to select a new character's race and pressing the "go left" button while on 'Human' still causes an Array-Out-of-Bounds exception.


I tossed together a quick fix that seems to work. (Would have done this yesterday, but I was just being lazy :P )

Replace:
_index = value < Items.Count || value > -1 ? value : 0;

With:
_index = value < Items.Count ? value < 0 ? Items.Count-1 : value : 0;


I tested it out and it seems to behave nicely, so hopefully we can call this one truly and completely resolved.
Main » Bug Reports » UIList Issue not quite resolved it totes is » New reply