Wedge

Public area => Bug reports => The Pub => Archived fixes => Topic started by: Arantor on August 1st, 2013, 06:16 AM

Title: Soft merge / ignored user
Post by: Arantor on August 1st, 2013, 06:16 AM
If you ignore a user and they have soft merged posts, both posts are collapsed as they should be but the second (or presumably third and onwards) softly merged post cannot be uncollapsed. What happens is that you click the link, it animates the uncollapse, then it animates recollapsing it. Tested here just now.
Title: Re: Soft merge / ignored user
Post by: Nao on August 1st, 2013, 06:39 PM
Hmmm.... Fixed locally. Not as elegantly as I'd like, but the logic of it... Well, couldn't do otherwise.
Title: Re: Soft merge / ignored user
Post by: Arantor on August 21st, 2013, 01:37 AM
Has this been committed? Haven't retested it yet.
Title: Re: Soft merge / ignored user
Post by: Nao on August 21st, 2013, 10:48 PM
I believe so. Around August 10, I think I'd committed all of my changes, really.

My current uncommitted stuff, and reason why:

- A warning message in Subs-Template due to constant not defined; because I don't know if it's best to test with defined() (as is the case on w.org), or to figure out why it happens that the skin option system is bypassed at some point, hmm...
- Some small layout tweaks; nothing important enough to hurry about, really.
- A new (important) mobile responsive layout tweak; applied on w.org, waiting for feedback until I commit. (There's been one so far, good.)
- Flag popup; not in a committable state, obviously...
- A Msg template rewrite; used here right now, no problems after multiple rewrites, so will commit soon.
- A fix in reqWin window height; waiting for a particular bug to occur again, so I can test it properly, really.
- A <wbr> fix; said bug only happened once, in one situation, and I'm actually not sure it's worth fixing, so...
- Fixed user boxes; waiting for commit, maybe with a skin option, don't know yet.
- extensive Warm rewrite; still not ready for prime time, has been in the works for months, now... :-/
Title: Re: Soft merge / ignored user
Post by: live627 on August 26th, 2013, 01:33 AM
Quote from Arantor on August 21st, 2013, 01:37 AM
Has this been committed? Haven't retested it yet.
I just tested it here on this site and the issue seems fixed.