Skip to content

Commit e88fbb6

Browse files
author
Greg Whitworth
committed
[resize-observer] Removed the inline issues that were resolved. Added a note about potential deprecation in the future of contentRect
1 parent c6d5391 commit e88fbb6

File tree

1 file changed

+2
-7
lines changed

1 file changed

+2
-7
lines changed

resize-observer-1/Overview.bs

+2-7
Original file line numberDiff line numberDiff line change
@@ -150,8 +150,6 @@ ResizeObserver can observe different kinds of CSS sizes:
150150
};
151151
</pre>
152152

153-
<p class='issue'>Is size the right name here? It's actually denoting which layout box we want to observe, possibly `box`?</p>
154-
155153
This section is non-normative. An author may desire to observe more than one CSS box.
156154
In this case, author will need to use multiple ResizeObservers.
157155

@@ -160,7 +158,7 @@ In this case, author will need to use multiple ResizeObservers.
160158
ro.observe(document.querySelector('#menu'), { box: 'content-box' });
161159

162160
// Observe just the border box. Replaces previous observation.
163-
ro.observe(document.querySelector('#menu'), { box: 'border-box' });
161+
ro1.observe(document.querySelector('#menu'), { box: 'border-box' });
164162
</pre>
165163

166164
<p class="note">This does not have any impact on which box dimensions are returned to the defined callback when the event is fired,
@@ -238,8 +236,7 @@ interface ResizeObserverEntry {
238236
};
239237
</pre>
240238

241-
<p class="issue">Do we only want to return the box(es) that were requested to be observed, or all? Leaning: all</p>
242-
<p class="issue">Do we want to add a note explaining that contentRect is for compatibility reasons, and might be depreciated? Leaning: yes</p>
239+
<p class="note">contentRect is from the incubation phase of ResizeObserver and is only included for current web compat reasons. It may be deprecated in future levels.</p>
243240

244241
<div dfn-type="attribute" dfn-for="ResizeObserverEntry">
245242
: <dfn>target</dfn>
@@ -290,8 +287,6 @@ interface ResizeObserverEntry {
290287
};
291288
</pre>
292289

293-
<p class="issue">We're covering the dimensions of the extra boxes, do we want to also cover their positions similiar to that of the initial contentRect?</p>
294-
295290
<h2 id="processing-model">Processing Model</h2>
296291

297292
<h3 id="resize-observation-interface">ResizeObservation example struct</h3>

0 commit comments

Comments
 (0)