Differential D12759

Hovercards - don't let them run off the right edge of the viewport anymore
ClosedPublic
Actions

Authored by btrahan on May 7 2015, 9:49 PM.

Tags

None

Referenced Files

	F18167424: D12759.id30669.diff
	Fri, Aug 15, 11:48 AM

	F18158018: D12759.diff
	Thu, Aug 14, 10:39 PM

	F17953594: D12759.id30675.diff
	Fri, Aug 1, 3:13 AM

	F17951245: D12759.id30668.diff
	Fri, Aug 1, 1:06 AM

	F17948956: D12759.id30669.diff
	Thu, Jul 31, 10:27 PM

	F17935876: D12759.diff
	Wed, Jul 30, 11:41 PM

	F17742971: D12759.id30675.diff
	Mon, Jul 21, 1:20 PM

	F17738297: D12759.diff
	Jul 20 2025, 11:22 PM

Subscribers

Details

Reviewers

Maniphest Tasks

T7524: Hovercards near screen edge should re-adjust placement

Commits

Restricted Diffusion Commit
rPf7c14736c173: Hovercards - don't let them run off the right edge of the viewport anymore

Summary

Fixes T7524.

Test Plan

made a task with a comment including another task. resized window so still desktop size and task reference on edge of window. invoked hovercard by bovering over task reference and noted the hovercard was completely visible.
opened the durable column and made a task reference. invoked hovercard by hovering over task reference and noted the hovercard was completely visible.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

btrahan updated this revision to Diff 30668.May 7 2015, 9:49 PM

btrahan added a task: T7524: Hovercards near screen edge should re-adjust placement.

btrahan retitled this revision from to Hovercards - don't let them run off the right edge of the viewport anymore.

btrahan updated this object.

btrahan edited the test plan for this revision. (Show Details)

btrahan added a reviewer: epriestley.

Herald added subscribers: epriestley, Korvin. · View Herald TranscriptMay 7 2015, 9:49 PM

add , 10 to parseInt call like the other cool kids

epriestley accepted this revision.May 7 2015, 10:59 PM

epriestley edited edge metadata.

epriestley added inline comments.

webroot/rsrc/js/core/Hovercard.js
102–107	Maybe toss this.

This revision is now accepted and ready to land.May 7 2015, 10:59 PM

Closed by commit rPf7c14736c173: Hovercards - don't let them run off the right edge of the viewport anymore (authored by btrahan, committed by btrahan). · Explain WhyMay 7 2015, 11:01 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

20 lines

webroot/

rsrc/

js/

core/

7 lines

Diff 30675

resources/celerity/map.php

Loading...

webroot/rsrc/js/core/Hovercard.js

Loading...