Page MenuHomePhabricator

Conpherence: Sent text floats below breadcrumb-bar
Closed, ResolvedPublic

Description

When I write text and send it, the page will refresh and display the newly sent text floating below the breadcrum-bar (i.e. the bar that displays Conpherence > remote-username). I assume after the recent adjustments (T7566#106681 and T5095) the message element is inserted into the DOM in the wrong place. The HTML looks like this: http://imgur.com/KMwzZ7y

Revisions and Commits

Restricted Differential Revision
Restricted Differential Revision

Event Timeline

devurandom raised the priority of this task from to Needs Triage.
devurandom updated the task description. (Show Details)
devurandom added a project: Conpherence.
devurandom added a subscriber: devurandom.

It's unrelated to T5095. Maybe scrollbar changes in D12396?

I can't seem to reproduce this, with or without a mouse plugged in. Comments send fine and scrolling up and down works too?

(With a mouse plugged in on production I am noticing auto-scrolling to bottom of a thread not working all that well, which is what D12396 did.)

I can repro so I'll take a look, I don't think I'm doing anything special (I have a mouse plugged in).

One possible issue is that (at least with Safari) you have to quit and relaunch the browser for it to change mouse/no-mouse state sometimes.

It looks like it's appending it on conpherence-messages, but not inside the viewport

pasted_file (121×647 px, 83 KB)

auto-scrolling to bottom of a thread not working all that well

I just wanted to report the same. This issue appears both when sending a message and when opening an existing conversation - in both cases the messages are not scrolled down all the way to the bottom.

epriestley added a revision: Restricted Differential Revision.Apr 14 2015, 7:53 PM
epriestley closed this task as Resolved by committing Restricted Diffusion Commit.Apr 14 2015, 8:04 PM
epriestley added a commit: Restricted Diffusion Commit.
epriestley added a revision: Restricted Differential Revision.Apr 14 2015, 9:55 PM
epriestley added a commit: Restricted Diffusion Commit.Apr 14 2015, 9:56 PM