Skip to content

Unify dart:html.{Rect,ClientRect,SVGRect} #3035

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
DartBot opened this issue May 14, 2012 · 12 comments
Closed

Unify dart:html.{Rect,ClientRect,SVGRect} #3035

DartBot opened this issue May 14, 2012 · 12 comments
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. closed-obsolete Closed as the reported issue is no longer relevant library-html P2 A bug or feature request we're likely to work on type-enhancement A request for a change that isn't a bug

Comments

@DartBot
Copy link

DartBot commented May 14, 2012

This issue was originally filed by @seaneagan


dart:html has three different interfaces to represent a Rectangle:

Rect
ClientRect
SVGRect

These ought to be unified into a single interface. The dart style guide recommends not to use abbreviations, so ideally it would be named 'Rectangle' instead of just 'Rect'.

@DartBot
Copy link
Author

DartBot commented May 14, 2012

This comment was originally written by antonm@google.com


cc @rakudrama.
cc @vsmenon.
Added Area-DOM label.

@sethladd
Copy link
Contributor

Removed Type-Defect label.
Added Type-Enhancement, Triaged labels.

@iposva-google
Copy link
Contributor

Removed Area-DOM label.
Added Area-HTML label.

@rakudrama
Copy link
Member

There is no field common to all three types.
ClientRect has 6 fields.
Rect has a subset, but they are CSSPrimitiveValues, not num-bers.
These types come from deep inside WekKit, so unifying them would require code to translate to and from the underlying representation.

I'm supportive of the general idea so we will revisit after M1 but I think it will turn out that these types are too different to unify.


Added this to the Later milestone.

@efortuna
Copy link
Contributor

putting as m7 to remind us to follow up with florian.


Removed this from the Later milestone.
Added this to the M7 milestone.

@efortuna
Copy link
Contributor

Set owner to @efortuna.

@efortuna
Copy link
Contributor

Added Waiting label.

@blois
Copy link

blois commented Oct 2, 2013

Removed this from the M7 milestone.
Added this to the M8 milestone.

@efortuna
Copy link
Contributor

efortuna commented Oct 4, 2013

In https://codereview.chromium.org//25808002 we moved Rectangle over to dart:math, and Client rect is merged with regular rect. SVG could probably be merged, but I'm postponing that part until a later milestone.


Removed this from the M8 milestone.
Added this to the Later milestone.
Added Accepted label.

@kevmoo
Copy link
Member

kevmoo commented Apr 7, 2014

Removed Area-HTML label.
Added Area-Library, Library-Html labels.

@kasperl
Copy link

kasperl commented Jul 10, 2014

Removed this from the Later milestone.
Added Oldschool-Milestone-Later label.

@kasperl
Copy link

kasperl commented Aug 4, 2014

Removed Oldschool-Milestone-Later label.

@DartBot DartBot added Type-Enhancement area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. library-html labels Aug 4, 2014
@kevmoo kevmoo added P2 A bug or feature request we're likely to work on type-enhancement A request for a change that isn't a bug and removed accepted labels Feb 29, 2016
@efortuna efortuna removed their assignment Nov 16, 2017
@matanlurey matanlurey added the closed-obsolete Closed as the reported issue is no longer relevant label Jun 19, 2018
copybara-service bot pushed a commit that referenced this issue May 20, 2022
…25 revisions)

https://dart.googlesource.com/dartdoc/+log/334072b0cad4..cf0685a2d1ce

2022-05-19 103135467+sealesj@users.noreply.github.com Create scorecards-analysis.yml (#3032)
2022-05-18 srawlins@google.com Add the docs.dart.js.map file to published package (#3042)
2022-05-17 parlough@gmail.com Reintroduce query param search (#3026)
2022-05-17 srawlins@google.com Include docs.dart.js in published code (#3040)
2022-05-17 parlough@gmail.com Optimize built js (#3028)
2022-05-17 srawlins@google.com Use dart run instead of deprecated dart pub run (#3038)
2022-05-17 timmaffett@gmail.com fix exception for empty dartdoc_options.yaml file (#3034)
2022-05-16 oss@simonbinder.eu Fix build config (#3035)
2022-05-02 goderbauer@google.com Drop version back to 5.1.0 (#3031)
2022-05-02 goderbauer@google.com Fixed: YouTube directive respects width/height argument (#3030)
2022-04-19 devoncarew@gmail.com Update pubspec.yaml (#3025)
2022-04-12 srawlins@google.com Remove dead field commentRefs, which is always empty (#3021)
2022-04-12 devoncarew@gmail.com make the converted web frontend follow Dart conventions (#3020)
2022-04-11 devoncarew@gmail.com convert lib/resources/script.js to Dart (#3016)
2022-04-11 49699333+dependabot[bot]@users.noreply.github.com Bump actions/cache from 3.0.1 to 3.0.2 (#3019)
2022-04-10 devoncarew@gmail.com rename the CI script (#3017)
2022-04-08 49699333+dependabot[bot]@users.noreply.github.com Bump actions/cache from 2 to 3.0.1 (#3014)
2022-04-08 49699333+dependabot[bot]@users.noreply.github.com Bump markdown from 4.0.1 to 5.0.0 (#3015)
2022-04-08 srawlins@google.com Move many static declarations to top-level; avoid RegExp (#3009)
2022-04-08 49699333+dependabot[bot]@users.noreply.github.com Bump actions/checkout from 2 to 3 (#3013)
2022-04-07 devoncarew@gmail.com Add a web frontend (#3011)
2022-04-07 devoncarew@gmail.com enable dependabot deps checks (#3012)
2022-04-07 srawlins@google.com Do not store the AstNode on ModelNode; eagerly grab the source indexes (#3008)
2022-04-07 srawlins@google.com Create a .pubignore in order to skip testing files from publish check (#3010)
2022-03-29 srawlins@google.com Bump to version 5.1.0 (#3006)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/dart-doc-dart-sdk
Please CC dart-ecosystem-gardener@grotations.appspotmail.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Dart Documentation Generator: https://github.com/dart-lang/dartdoc/issues
To file a bug in Dart SDK: https://github.com/dart-lang/sdk/issues

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Tbr: dart-ecosystem-gardener@grotations.appspotmail.com
Change-Id: I3467a27d6947c43f7f41acbb2959d72863fd8a34
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/245483
Commit-Queue: Devon Carew <devoncarew@google.com>
Reviewed-by: Devon Carew <devoncarew@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. closed-obsolete Closed as the reported issue is no longer relevant library-html P2 A bug or feature request we're likely to work on type-enhancement A request for a change that isn't a bug
Projects
None yet
Development

No branches or pull requests

9 participants