Skip to content
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

Reference list of missing issues for use cases and capabilities #110

Closed
1 of 12 tasks
Malvoz opened this issue Aug 2, 2019 · 22 comments
Closed
1 of 12 tasks

Reference list of missing issues for use cases and capabilities #110

Malvoz opened this issue Aug 2, 2019 · 22 comments
Labels
meta/process An issue related to how the work or repo is managed.

Comments

@Malvoz
Copy link
Member

Malvoz commented Aug 2, 2019

In the hope of potentially saving you guys some time, I've been going through the spec for use cases and capabilities that do not link to an issue for discussion, and I've also tried to find the corresponding issue in any case there is one but it just hasn't been linked to from the spec.


Edit: Things TBD as listed below have been updated to reflect changes in:
0071b30, ae59ecb, 6826d37, #124, #162.


These examples have broken links to corresponding use cases in the spec:


These issues aren't properly linking to their corresponding section in the spec, e.g.:

This issue is for discussion of <link>.

is missing, or have a broken link:

Items in #110 (comment)


These use cases and capabilities are currently not linking to a corresponding github issue (perhaps because no such issue exist yet):

@AmeliaBR
Copy link
Member

AmeliaBR commented Aug 2, 2019

Yes, this is the intention. It has been on my To-do for a while to go through & find the missing cross-links. So thank you for the reference list! I will try to get to these this week.

@AmeliaBR AmeliaBR added the meta/process An issue related to how the work or repo is managed. label Aug 2, 2019
@Malvoz Malvoz changed the title [meta/process] Is the intention that all use cases and capabilities have their own issue? Reference list of missing issues for use cases and capabilities Aug 4, 2019
@Malvoz

This comment has been minimized.

AmeliaBR added a commit that referenced this issue Aug 9, 2019
- Introduces a standard example template,
  with a ToC and entries for every reference JS library.
- Makes existing example pages match,
  and adds stub pages where they didn't exist.
- Merges a few duplicate example pages from the merger of use cases.
- Renames every example page to be shorter:
   don't include "use case" to start, match new section IDs,
   shorten some section IDs
   (note, some of the renames are showing up as delete/add, but oh well)
- Make sure cross-references to example pages are correct.
- Make sure cross-references to issues are correct.

Also includes some new use case text,
because I forgot to commit that before I got side-tracked
working on the examples!

Still to do: the same thing for the rest of the use cases!

Related to #110 and #100
@Malvoz
Copy link
Member Author

Malvoz commented Aug 10, 2019

@AmeliaBR I've updated the lists above to reflect the changes made in 0071b30. And regarding the (new) examples that link back to use cases, many of the links are broken because they don't include some determiners while the ids for the use case sections do. For example:

The show-vector-data-overlay-map.html example links to #use-case-show-vector-data-overlay-map while the id for that section is use-case-show-a-vector-data-overlay-on-a-map. Would you rather change the ids in the spec, or the links from the examples in these cases?

@AmeliaBR
Copy link
Member

Thanks for the updates @Malvoz. Yes, I'm still working on going through the rest of the links; that commit was just the first batch. But… it looks like I messed up in a few cases, by not including the use-case- prefix on the link back to the spec from the example page. I'll review that.

I've also been simplifying the IDs as I go (to make it less likely that these little discrepancies will slip through), so for your example, probably both of those will end up changed!

Once I've done a full pass, I'll run the whole thing through a link checker & make sure we get this all tidied up.

@AmeliaBR
Copy link
Member

OK, I think I've fixed the broken links and filled in the missing discussion issues for all the use-case sections (as of ae59ecb).

Ran the main document through the W3C link checker, so at least I am sure that the links out of the report are OK. (Well, except for links to GitHub, since the link checker is blocked by robots.txt! But it's more likely that those are valid links but to the wrong issue rather than dead links, anyway!)

I still need to do another pass for editorial/content clean-up and standardizing the examples, so I'll double check the back-references (from the example files to the report) at that time.

Next step: missing issues for the capabilities section.
Also: avoid breaking things again as we fill in additional sections!

@Malvoz
Copy link
Member Author

Malvoz commented Aug 10, 2019

Thanks, I've updated all the lists! :)

I resorted to removing things in the checklists that have been finished instead of checking the boxes, for an easier overview of what needs TBD.

@Malvoz
Copy link
Member Author

Malvoz commented Aug 26, 2019

The list has been updated per changes in #124.

@Malvoz
Copy link
Member Author

Malvoz commented Sep 6, 2019

These use cases and capabilities are currently not linking to a corresponding github issue (perhaps because no such issue exist yet):

@AmeliaBR

I'd like to open issues for all the cases where there is none yet, and also cross reference them from the spec afterwards. Is that okay with you?

I'm not permitted to label issues - so I could add a follow-up comment on which labels for you to set to save you time, then I'd delete those comments at any time you've set the corresponding label(s).

@AmeliaBR
Copy link
Member

AmeliaBR commented Sep 8, 2019

Hi @Malvoz, If you want to go ahead and make the rest of the missing discussion pages, I would be grateful. I haven't had nearly as much time & energy for this project as I'd like this past month, and your contributions are definitely appreciated.

I've granted you Triage permissions for the repo, so you should be able to add any labels as required. But if you're not sure about any of the details, just ping me in a comment.

@nchan0154
Copy link
Contributor

Seconding the gratitude, @Malvoz! I have been meaning to go through these but I'm dealing with a bad case of RSI at the moment and repetitive clicking actions like making issues en-masse are difficult. Your assistance here would be much appreciated!

@Malvoz
Copy link
Member Author

Malvoz commented Sep 9, 2019

@AmeliaBR "Take my energy ༼ つ ◕_◕ ༽つ" 😄

Thanks for allowing me to do this!

And thank you @nchan0154 for the kind words, too!

@AmeliaBR
Copy link
Member

AmeliaBR commented Sep 9, 2019

Whoosh!

I admit, I had one moment of panic when I saw the 30+ notification emails. Then I realized what they all were & “Hooray!”

Thanks again. It is very great to have those links ready before the big TPAC meeting, when we'll be trying to get new people looking at the report & starting discussions.

@Malvoz
Copy link
Member Author

Malvoz commented Sep 10, 2019

@AmeliaBR, @prushforth, @nchan0154, @NickFitz
When you find that you have some time... (although this shouldn't take very long)

I think it's important (particularly for newcomers) that the issues for discussion properly link to their respective section in the report. Since refactoring some time ago, quite a few of the URLs have changed for certain sections of the report, thus they'll need to be updated - in some cases there is no link at all.

But this is all done below, so as issue owners you'll just have to edit your initial comment in each of those issues and copy & paste to include at the top:

@AmeliaBR's

Done!

Issue #12:

This issue is for discussion of <a href="https://maps4html.github.io/HTML-Map-Element-UseCases-Requirements/#mapbox">the reference tool "Mapbox GL JS API and Mapbox Studio embed"</a>.
<hr>

Issue #13:

This issue is for discussion of <a href="https://maps4html.github.io/HTML-Map-Element-UseCases-Requirements/#d3-geo">the reference tool "D3 Geographies APIs"</a>.
<hr>

Issue #26:

This issue is for discussion of <a href="https://maps4html.github.io/HTML-Map-Element-UseCases-Requirements/#openstreetmap">the reference tool "OpenStreetMap embeds"</a>.
<hr>

Issue #27:

This issue is for discussion of <a href="https://maps4html.github.io/HTML-Map-Element-UseCases-Requirements/#bing-maps">the reference tool "Bing Maps embeds and Bing Maps Control API"</a>.
<hr>

Issue #28:

This issue is for discussion of <a href="https://maps4html.github.io/HTML-Map-Element-UseCases-Requirements/#mapkit-js">the reference tool "MapKit JS (Apple Maps) API"</a>.
<hr>

Issue #29:

This issue is for discussion of <a href="https://maps4html.github.io/HTML-Map-Element-UseCases-Requirements/#leaflet-js">the reference tool "Leaflet.js API"</a>.
<hr>

Issue #30:

This issue is for discussion of <a href="https://maps4html.github.io/HTML-Map-Element-UseCases-Requirements/#openlayers">the reference tool "OpenLayers API"</a>.
<hr>

@prushforth's

Done!

Issue #3:

This issue is for discussion of <a href="https://maps4html.github.io/HTML-Map-Element-UseCases-Requirements/#capability-reproject-maps">the map viewer capability "Reproject map tile data into a new projection or globe view"</a>.
<hr>
<!-- 
Please also change the title of this issue to:
"Capability: Reproject map tile data into a new projection or globe view"
-->

Issue #9:

This issue is for discussion of <a href="https://maps4html.github.io/HTML-Map-Element-UseCases-Requirements/#google-maps">the reference tool "Google Maps embeds and Google Maps Platform API"</a>.
<hr>

Issue #14:

This issue is for discussion of <a href="https://maps4html.github.io/HTML-Map-Element-UseCases-Requirements/#capability-tile-coordinate-systems">the map viewer capability "Display map tiles defined in various common coordinate systems"</a>.
<hr>
<!--
Please also change the title of this issue to:
"Capability: Display map tiles defined in various common coordinate systems"
-->

Issue #15:

This issue is for discussion of <a href="https://maps4html.github.io/HTML-Map-Element-UseCases-Requirements/#capability-styled-features">the map viewer capability "Apply custom styling to map markers and vector features"</a>.
<hr>
<!--
Please also change the title of this issue to:
"Capability: Apply custom styling to map markers and vector features"
-->

Issue #16:

This issue is for discussion of <a href="https://maps4html.github.io/HTML-Map-Element-UseCases-Requirements/#capability-custom-controls">the map viewer capability "Toggle whether default controls are displayed"</a>.
<hr>
<!--
Please also change the title of this issue to:
"Capability: Toggle whether default controls are displayed"
-->

Issue #21:

Please change the title of this issue to:
"Access additional information about a point on the map".


@NickFitz's

Done!

Issue #50:

This issue is for general discussions of <a href="https://maps4html.github.io/HTML-Map-Element-UseCases-Requirements/#client-apis">the "Client-side Mapping API Capabilities"</a>.
<hr>

Issue #55:

This issue is for discussion of <a href="https://maps4html.github.io/HTML-Map-Element-UseCases-Requirements/#capability-create-initialise-and-display-a-map">the API capability "Create, initialise, and display a map"</a>.
<hr>

Issue #57:

This issue is for discussion of <a href="https://maps4html.github.io/HTML-Map-Element-UseCases-Requirements/#capability-implement-a-custom-control">the API capability "Implement a custom control"</a>.
<hr>

Issue #74:

This issue is for discussion of <a href="https://maps4html.github.io/HTML-Map-Element-UseCases-Requirements/#capability-define-data-source-tile-layer">the required API capability "Define a data source for a tile layer"</a>.
<hr>

Issue #81:

This issue is for discussion of <a href="https://maps4html.github.io/HTML-Map-Element-UseCases-Requirements/#use-case-alternative-layers">the use case "Provide alternative map layers which the user can select"</a>.
<hr>

@nchan0154's

Done!

Issue #49:

This issue is for discussion of <a href="https://maps4html.github.io/HTML-Map-Element-UseCases-Requirements/#tomtom">the reference tool "TomTom Maps SDK for Web"</a>.
<hr>

@NickFitz
Copy link
Contributor

@Malvoz I've just done mine, including editing the title for a couple of them to match the section they refer to where the wording of that's changed.

Thanks for your work on this!

@Malvoz
Copy link
Member Author

Malvoz commented Sep 10, 2019

@NickFitz

Thanks!

I had actually missed updating the link for #74, I've updated it now, could you redo that one?

@NickFitz
Copy link
Contributor

@Malvoz No problem, done :-)

@nchan0154
Copy link
Contributor

Done, thank you @Malvoz!

@prushforth
Copy link
Member

Thank you Robert, you are a great collaborator! Done.

@prushforth
Copy link
Member

I did @AmeliaBR's too, cause she's either busy or asleep. I hope that's not bad github etiquette? I was on a roll in an airport lounge...

@Malvoz
Copy link
Member Author

Malvoz commented Sep 13, 2019

Thanks Peter!

Regarding issue #21 - I had missed to include "Use Case:" in its title, it should rather be:
"Use case: Access additional information about a point on the map".

And in #9 you can remove the outdated reference:

Relevant section: https://maps4html.github.io/HTML-Map-Element-UseCases-Requirements/#google-maps-embeds-and-google-maps-platform-api


now that you've added the new reference.

Again, thanks, for doing both yours and Amelia's. This is great! Safe travels! 😄

@AmeliaBR
Copy link
Member

Thanks Peter & Robert!

When I came up with this idea of using GitHub issues for section-by-section discussion, I did not think about all the extra work of keeping them in sync as the document gets re-written and re-organized!

@Malvoz
Copy link
Member Author

Malvoz commented Oct 7, 2019

The example pages have been revamped some time ago, and will need more work but I no longer consider them (or them not linking properly to the spec) "broken", closing this. Thanks for all your help! :)

@Malvoz Malvoz closed this as completed Oct 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
meta/process An issue related to how the work or repo is managed.
Projects
None yet
Development

No branches or pull requests

5 participants