Page MenuHomePhabricator
Feed Advanced Search

May 21 2021

epriestley added a comment to T13451: Chrome double-click selection behavior has changed in LayoutNG.

See PHI2090 for another report of this. Chrome hasn't changed behavior since the last update, so I'm more inclined to look at workarounds.

May 21 2021, 6:28 PM · Google Chrome

Sep 28 2020

epriestley added a revision to T13564: Remarkup tables should show all content when printed to PDF: D21472: When printing, wrap all content in Remarkup tables more aggressively.
Sep 28 2020, 4:22 PM · Google Chrome, Remarkup

Aug 11 2020

epriestley added a comment to T13564: Remarkup tables should show all content when printed to PDF.

Adding overflow-wrap: anywhere appears to fix this without breaking anything.

Aug 11 2020, 5:35 PM · Google Chrome, Remarkup
epriestley triaged T13564: Remarkup tables should show all content when printed to PDF as Normal priority.
Aug 11 2020, 5:17 PM · Google Chrome, Remarkup

Nov 8 2019

epriestley added a comment to T13451: Chrome double-click selection behavior has changed in LayoutNG.

I'm planning to give Chrome some time to triage the report. If it sits there for a while or they decide it's how LayoutNG is going to handle this case, I'll look for workarounds. I don't think our intent/markup is totally unambiguous, and it may be reasonable to decide that this behavior is acceptable, even though Safari and Firefox have different behavior.

Nov 8 2019, 4:10 PM · Google Chrome
epriestley updated the task description for T13451: Chrome double-click selection behavior has changed in LayoutNG.
Nov 8 2019, 4:07 PM · Google Chrome
epriestley renamed T13451: Chrome double-click selection behavior has changed in LayoutNG from Chrome double-click selection behavior has changed in Chrome ~78? to Chrome double-click selection behavior has changed in LayoutNG.
Nov 8 2019, 4:06 PM · Google Chrome
epriestley added a comment to T13451: Chrome double-click selection behavior has changed in LayoutNG.

Here's a simple reproduction case:

Nov 8 2019, 4:05 PM · Google Chrome
epriestley triaged T13451: Chrome double-click selection behavior has changed in LayoutNG as Low priority.
Nov 8 2019, 3:41 PM · Google Chrome

Sep 24 2019

epriestley lowered the priority of T13413: Chrome 77 rendering engine freezes when elements are nested with particular word-wrap / white-space CSS behavior from Normal to Wishlist.

This now appears to be fixed in the release version of Chrome. We can remove the workaround in some future change.

Sep 24 2019, 6:10 PM · Google Chrome

Sep 13 2019

epriestley added a revision to T13413: Chrome 77 rendering engine freezes when elements are nested with particular word-wrap / white-space CSS behavior: D20812: Work around rendering engine freeze in Chrome 77 affecting workboards.
Sep 13 2019, 1:59 AM · Google Chrome

Sep 11 2019

epriestley added a comment to T13413: Chrome 77 rendering engine freezes when elements are nested with particular word-wrap / white-space CSS behavior.

I upstreamed this to Chrome here: https://bugs.chromium.org/p/chromium/issues/detail?id=1003002

Sep 11 2019, 6:00 PM · Google Chrome
epriestley triaged T13413: Chrome 77 rendering engine freezes when elements are nested with particular word-wrap / white-space CSS behavior as Normal priority.
Sep 11 2019, 5:53 PM · Google Chrome

Aug 6 2017

chad closed T8122: Unable to scroll on mobile if horizontal scroll is available as Invalid.

No longer valid as far as I can find.

Aug 6 2017, 3:52 AM · Google Chrome, PHUI, Mobile

Jul 9 2017

chad removed the image for Google Chrome.
Jul 9 2017, 7:35 PM

Jan 12 2017

epriestley added a comment to T12032: Make sure Chrome 55 anchor issue does not impact Differential.

Chrome 56 has merged a fix for this so it should be moot in the not-too-distant future, although I'm not exactly sure what the Chrome release schedule looks like. If this only impacts ghost inlines for a few more weeks it's pretty tempting to just wait for it to resolve itself, but I may be digging around in that code soon so perhaps I'll take a more detailed look.

Jan 12 2017, 9:06 PM · Bug Report (Needs Information), Phacility, Google Chrome, Differential
chris.siden added a comment to T12032: Make sure Chrome 55 anchor issue does not impact Differential.

Is it possible that this issue effects Differential but only for inline-comments that have been ported forward from a previous diff (i.e. the ones that are rendered as slightly faded). I see this issue on this instance with this revision: https://secure.phabricator.com/D15675, none of the anchor links in the timeline work for me when I use Chrome 55, but I don't see the same issue on reviews where comments are on the most-recent diff (not faded).

Jan 12 2017, 8:53 PM · Bug Report (Needs Information), Phacility, Google Chrome, Differential

Dec 28 2016

epriestley closed T12032: Make sure Chrome 55 anchor issue does not impact Differential as Resolved.

We haven't received more information about this in about a week and can't reproduce it, so we can't move forward. I'm going to close this until we get new information.

Dec 28 2016, 4:31 PM · Bug Report (Needs Information), Phacility, Google Chrome, Differential

Dec 22 2016

epriestley added a project to T12032: Make sure Chrome 55 anchor issue does not impact Differential: Bug Report (Needs Information).
Dec 22 2016, 2:04 PM · Bug Report (Needs Information), Phacility, Google Chrome, Differential

Dec 20 2016

epriestley added a project to T12032: Make sure Chrome 55 anchor issue does not impact Differential: Phacility.
Dec 20 2016, 1:00 PM · Bug Report (Needs Information), Phacility, Google Chrome, Differential
epriestley added a comment to T12032: Make sure Chrome 55 anchor issue does not impact Differential.

I wasn't immediately able to reproduce this:

Dec 20 2016, 1:00 PM · Bug Report (Needs Information), Phacility, Google Chrome, Differential

Dec 19 2016

epriestley created T12032: Make sure Chrome 55 anchor issue does not impact Differential.
Dec 19 2016, 9:35 PM · Bug Report (Needs Information), Phacility, Google Chrome, Differential

Jul 28 2016

epriestley closed T11386: Weird effect on closed tasks in graph view as Resolved by committing rPcebf4bbec6ba: In Task Graphs, provide a parent/child hint and fix weird strikethrough.
Jul 28 2016, 6:50 PM · Google Chrome, Bug Report
epriestley added a revision to T11386: Weird effect on closed tasks in graph view: D16342: In Task Graphs, provide a parent/child hint and fix weird strikethrough.
Jul 28 2016, 6:48 PM · Google Chrome, Bug Report
sshannin added a comment to T11386: Weird effect on closed tasks in graph view.

No biggie, although the UI effect is obviously crucial for my org

Jul 28 2016, 12:40 AM · Google Chrome, Bug Report
chad reopened T11386: Weird effect on closed tasks in graph view as "Open".

Whoops, yeah the strikethrough isn't for some reason. Sorry I thought you meant an issue with the graph itself.

Jul 28 2016, 12:34 AM · Google Chrome, Bug Report
chad merged task T11386: Weird effect on closed tasks in graph view into T4788: Allow "Edit Dependencies" both ways (blocking and depending).
Jul 28 2016, 12:33 AM · Google Chrome, Bug Report
sshannin created T11386: Weird effect on closed tasks in graph view.
Jul 28 2016, 12:32 AM · Google Chrome, Bug Report

Jun 30 2016

epriestley closed T11238: Hello as Spite.
Jun 30 2016, 10:44 AM · Google Chrome, Phabot, Bug Report
kingbonio added a comment to T11238: Hello.

I've located the mainframe and uploaded the downloads

Jun 30 2016, 9:04 AM · Google Chrome, Phabot, Bug Report
kingbonio created T11238: Hello.
Jun 30 2016, 9:03 AM · Google Chrome, Phabot, Bug Report

Feb 20 2016

ftdysa added a comment to T10396: Remarkup toolbar actions always cause comment box to scroll to bottom.

Ah cool, good to know. Thanks for the quick fix!

Feb 20 2016, 12:15 AM · Remarkup, Google Chrome, Bug Report

Feb 19 2016

epriestley closed T10396: Remarkup toolbar actions always cause comment box to scroll to bottom as Resolved by committing rP7b1b146620b5: Save, then restore scroll position in Chrome textareas on remarkup assist.
Feb 19 2016, 10:55 PM · Remarkup, Google Chrome, Bug Report
epriestley added a comment to T10396: Remarkup toolbar actions always cause comment box to scroll to bottom.

Yeah, I came to approximately the same conclusion. Save + restore on the scroll position appears to be a complete fix as far as I can tell.

Feb 19 2016, 10:55 PM · Remarkup, Google Chrome, Bug Report
epriestley added a revision to T10396: Remarkup toolbar actions always cause comment box to scroll to bottom: D15313: Save, then restore scroll position in Chrome textareas on remarkup assist.
Feb 19 2016, 10:49 PM · Remarkup, Google Chrome, Bug Report
ftdysa added a comment to T10396: Remarkup toolbar actions always cause comment box to scroll to bottom.

So I looked into this and it is the call to area.focus() in the setSelectionRange method in https://secure.phabricator.com/diffusion/P/browse/master/webroot/rsrc/js/core/TextAreaUtils.js (unsure what line, my view of this file is plaintext here) that causes the scrolling. I removed this line and it seemed to have the expected behavior but who knows what other problems removing it may cause.

Feb 19 2016, 10:19 PM · Remarkup, Google Chrome, Bug Report
chad added projects to T10396: Remarkup toolbar actions always cause comment box to scroll to bottom: Google Chrome, Remarkup.
Feb 19 2016, 9:25 PM · Remarkup, Google Chrome, Bug Report

Feb 2 2016

chad updated the image for Google Chrome from F1064814: profile to F1087336: profile.
Feb 2 2016, 4:54 PM

Jan 19 2016

chad set the image for Google Chrome to F1064814: profile.
Jan 19 2016, 10:07 PM

Oct 22 2015

epriestley added a comment to T9612: Unable to scroll after clicking on inline comment link in Chrome 46.

Yeah, I have the same guess as @chad on this.

Oct 22 2015, 1:57 PM · Differential, Google Chrome
oujesky closed T9612: Unable to scroll after clicking on inline comment link in Chrome 46 as Invalid.

Yes, you are right, we are on version from April 2015. Sorry for that.

Oct 22 2015, 1:57 PM · Differential, Google Chrome
chad added a comment to T9612: Unable to scroll after clicking on inline comment link in Chrome 46.

I presume you are behind D12789, which was May 2015.

Oct 22 2015, 1:53 PM · Differential, Google Chrome
oujesky added a comment to T9612: Unable to scroll after clicking on inline comment link in Chrome 46.

I tried to reproduce the issue here, but it looks fine (not sure which version is deployed here). I tried on https://secure.phabricator.com/D14297#inline-48363 and I can scroll with every means.

Oct 22 2015, 1:53 PM · Differential, Google Chrome
epriestley added a comment to T9612: Unable to scroll after clicking on inline comment link in Chrome 46.

This does not reproduce for me with Chrome 46 on Windows 7.

Oct 22 2015, 1:52 PM · Differential, Google Chrome
oujesky added a comment to T9612: Unable to scroll after clicking on inline comment link in Chrome 46.

Tried on Windows 7 and Windows 10.

Oct 22 2015, 1:45 PM · Differential, Google Chrome
chad added a comment to T9612: Unable to scroll after clicking on inline comment link in Chrome 46.

Mac Desktop / Mouse / Chrome 46 also works fine. @oujesky what OS are you running and is your Phabricator at HEAD?

Oct 22 2015, 1:42 PM · Differential, Google Chrome
epriestley added a comment to T9612: Unable to scroll after clicking on inline comment link in Chrome 46.

I also can't reproduce this with a similar setup (OSX / Macbook / Chrome 46).

Oct 22 2015, 1:32 PM · Differential, Google Chrome
chad added a comment to T9612: Unable to scroll after clicking on inline comment link in Chrome 46.

I gave a quick test on a laptop / Mac / Chrome 46 and can't really reproduce anything on https://secure.phabricator.com/D14268#inline-48344. @epriestley can you?

Oct 22 2015, 1:31 PM · Differential, Google Chrome
epriestley added a comment to T9612: Unable to scroll after clicking on inline comment link in Chrome 46.

See T7740 / https://code.google.com/p/chromium/issues/detail?id=473657 for a previous adventure with scrolling Chrome anchors.

Oct 22 2015, 1:11 PM · Differential, Google Chrome
oujesky created T9612: Unable to scroll after clicking on inline comment link in Chrome 46.
Oct 22 2015, 10:57 AM · Differential, Google Chrome

May 14 2015

epriestley closed T7740: Chrome exhibits weird scrolling jumping with anchor in the URL as Resolved.

This should be resolved in HEAD because we no longer frame the page content. Chrome still has the underlying bug.

May 14 2015, 11:07 PM · Google Chrome

May 10 2015

epriestley moved T8122: Unable to scroll on mobile if horizontal scroll is available from Backlog to Scrollbar on the PHUI board.
May 10 2015, 4:39 PM · Google Chrome, PHUI, Mobile

May 7 2015

chad added a project to T8122: Unable to scroll on mobile if horizontal scroll is available: Google Chrome.
May 7 2015, 9:20 PM · Google Chrome, PHUI, Mobile

Apr 17 2015

bkuhn added a comment to T7740: Chrome exhibits weird scrolling jumping with anchor in the URL.

In case it helps--calling window.scrollTo(window.scrollX, window.scrollY) right after I hit the back button appears to get my browser out of the broken state from the second repro (first one no longer works for me). Might it work to call that at various points when Chrome might have entered a broken state?

Apr 17 2015, 11:36 PM · Google Chrome
epriestley added a comment to T7740: Chrome exhibits weird scrolling jumping with anchor in the URL.

The only workarounds I can come up with are:

Apr 17 2015, 11:29 PM · Google Chrome
chad added a comment to T7740: Chrome exhibits weird scrolling jumping with anchor in the URL.

We should probably investigate a workaround, as any Chrome fix is months away, even when acknowledged.

Apr 17 2015, 11:27 PM · Google Chrome
chad updated subscribers of T7740: Chrome exhibits weird scrolling jumping with anchor in the URL.
Apr 17 2015, 11:24 PM · Google Chrome

Apr 12 2015

chad merged T7806: Viewing diff with anchor in url makes page jumpy into T7740: Chrome exhibits weird scrolling jumping with anchor in the URL.
Apr 12 2015, 1:12 AM · Google Chrome
chad merged task T7806: Viewing diff with anchor in url makes page jumpy into T7740: Chrome exhibits weird scrolling jumping with anchor in the URL.
Apr 12 2015, 1:12 AM · Google Chrome, Diffusion
Mnkras created T7806: Viewing diff with anchor in url makes page jumpy.
Apr 12 2015, 1:03 AM · Google Chrome, Diffusion

Apr 7 2015

epriestley removed a parent task for T7740: Chrome exhibits weird scrolling jumping with anchor in the URL: T5042: Futures()->limit(X) does not ensure that X number of tasks are running after the first set of tasks.
Apr 7 2015, 12:25 PM · Google Chrome
nagym718 added a parent task for T7740: Chrome exhibits weird scrolling jumping with anchor in the URL: T5042: Futures()->limit(X) does not ensure that X number of tasks are running after the first set of tasks.
Apr 7 2015, 8:04 AM · Google Chrome

Apr 3 2015

epriestley renamed T7740: Chrome exhibits weird scrolling jumping with anchor in the URL from Weird scrolling jumping with anchor in the URL to Chrome exhibits weird scrolling jumping with anchor in the URL.
Apr 3 2015, 1:36 PM · Google Chrome

Mar 10 2015

chad closed T7032: Scrolling of revisions is broken on Chrome 41 all as Resolved.

Presume fixed since you don't to scroll left/right anymore.

Mar 10 2015, 10:44 PM · Google Chrome, Differential, Mobile

Mar 5 2015

avivey added a comment to T7032: Scrolling of revisions is broken on Chrome 41 all.

Just tested it again with a touchpad (laptop), and it still works.

Mar 5 2015, 5:50 PM · Google Chrome, Differential, Mobile
chad added a comment to T7032: Scrolling of revisions is broken on Chrome 41 all.

Are you on a laptop? Seems touch specific.

Mar 5 2015, 2:24 PM · Google Chrome, Differential, Mobile
avivey added a comment to T7032: Scrolling of revisions is broken on Chrome 41 all.

works for me on Windows/Chrome 41. Just to make it more annoying, I guess.

Mar 5 2015, 6:06 AM · Google Chrome, Differential, Mobile
chad added a comment to T7032: Scrolling of revisions is broken on Chrome 41 all.

Well this now repros in plain Mac/Chrome v41. Help me 1-up diff view, you're my only hope!

Mar 5 2015, 4:20 AM · Google Chrome, Differential, Mobile
chad renamed T7032: Scrolling of revisions is broken on Chrome 41 all from Scrolling of revisions is broken on Chrome 40 on Android to Scrolling of revisions is broken on Chrome 41 all.
Mar 5 2015, 4:19 AM · Google Chrome, Differential, Mobile

Jan 25 2015

epriestley added a comment to T7033: Keyboard-based browsing of Diffs/Commits/Revisions is broken.

This should be fixed in HEAD, and on this server. Let us know if you still run into issues. Thanks for the report!

Jan 25 2015, 4:47 PM · Google Chrome
epriestley closed T7033: Keyboard-based browsing of Diffs/Commits/Revisions is broken as Resolved by committing rPea67a8ab8e58: Make scrolling happen relative to the main content frame.
Jan 25 2015, 4:42 PM · Google Chrome
epriestley added a revision to T7033: Keyboard-based browsing of Diffs/Commits/Revisions is broken: D11490: Make scrolling happen relative to the main content frame.
Jan 25 2015, 1:42 PM · Google Chrome
johnny-bit added a comment to T7033: Keyboard-based browsing of Diffs/Commits/Revisions is broken.

I can confirm same behaviour on Firefox 35 on linux when browsing for example D11489

Jan 25 2015, 10:37 AM · Google Chrome
epkugelmass added a project to T7033: Keyboard-based browsing of Diffs/Commits/Revisions is broken: Google Chrome.
Jan 25 2015, 8:37 AM · Google Chrome