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

Rolling Doc Changes July 2020 #26

Merged
merged 46 commits into from
Dec 17, 2020
Merged

Rolling Doc Changes July 2020 #26

merged 46 commits into from
Dec 17, 2020

Conversation

IronRico
Copy link
Contributor

I added wording to detail the WGS 84 requirements to the Extent Import Features page. A link to an outside resource was also made available in the description as well.

"{% hint style='info' %}
Please Note: Imported data needs to utilize the widely used World Geodetic System 1984 (WGS 84) [WGS84] coordinate reference system. This has longitude and latitude set in units of decimal degrees. For more details on this, please visit the espg.io coordinate support site. Here you can find details for the defined unit measurements as well as popular file export options.
{% endhint %}"

Carlos.P

I added wording to detail the WGS 84 requirements to the Extent Import Features page.  A link to an outside resource was also made available in the description as well.

"{% hint style='info' %}
**Please Note:**  Imported data needs to utilize the widely used World Geodetic System 1984 (WGS 84) [WGS84] coordinate reference system.  This has longitude and latitude set in units of decimal degrees. For more details on this, please visit <a href="https://epsg.io/4326" target="_blank">the espg.io coordinate support site</a>.  Here you can find details for the defined unit measurements as well as popular file export options.
{% endhint %}"


  Carlos.P
IronRico added 14 commits June 15, 2020 18:09
As the sidebar highlight function is removed, this small update simply removes mentions of the feature from the page.  I also added wordage to advise on the matter of possible screen size issues.

"The feature is made to work with the user’s native screen size.  Note, that on some smaller screens the sidebar may place itself in a fixed position to accommodate the display area. "
Added a new screenshot for temp use to test out some ideas.  For use with the secondary sidebar portion of the mdEditor guide found at this current location.  

https://guide.mdeditor.org/tutorial/editor-window-parts/secondary-sidebar/fast-scrolling.html
New screenshot for use with fast scrolling page, noting lack of highlight feature on sidebar.
Simple deletion.  Placed in wrong area and still needs more work.
Added a new screenshot for the fast scrolling page.
Added a new screenshot for use with fast scrolling page.  Noted that it seems larger on display and will need some edits to be more in line with the mdEditor style.
I uploaded a screenshot for the "Online Transfer" option for use in the "Distribution" section.
I created a new page for the "Online Transfer Option" section for use with the "Distribution" section.
I created a new section under distribution to have the "Online Transfer Option" setup as it's own page.
Copy link
Member

@jlblcc jlblcc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See inline code suggestions.

Copy link
Member

@jlblcc jlblcc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See inline code suggestions.

jlblcc
jlblcc previously requested changes Jun 30, 2020
Copy link
Member

@jlblcc jlblcc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See inline code suggestions.

IronRico and others added 8 commits July 12, 2020 15:47
Per suggestion, replaced file contents with an "include" statement.  Tested locally.
Removed stars that were added by mistake.
Added the online transfer option markdown file here for use as an include object.
Removed the bottom section of the file and added links within the main page area.
Updated file to have title of "Transfer Frequency".
@IronRico
Copy link
Contributor Author

For this issue: "Remove "Attribute" from "Primary/Foreign" elements #33"

I used a new screenshot. Please review for any considered issues:


AtrributeNameAnnotations

This file was deleted as it was both a duplicate file and also in the wrong path.
Edited the following links on the Summary,md file to allow for proper navigation:

[Online Transfer Option](reference/edit-window/metadata/distribution-panels/ onlineTransferOption.md)
[Transfer Frequency] (reference/edit-window/metadata/distribution-panels/ timeDuration-obj.md)
@IronRico
Copy link
Contributor Author

IronRico commented Aug 4, 2020

I worked to correct the navigation issues found in the Distribution section of the Summary.md file.

I replaced these with the following:

  • [Online Transfer Option](reference/edit-window/metadata/distribution-panels/ onlineTransferOption.md)
  • [Transfer Frequency] (reference/edit-window/metadata/distribution-panels/ timeDuration-obj.md)

I tested these items locally and found them to be working as intended.

Copy link
Contributor Author

@IronRico IronRico left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes proposed were accepted.

@IronRico
Copy link
Contributor Author

IronRico commented Aug 4, 2020

I reviewed and commented on the changes requested. None were declined with all being implemented.

@IronRico
Copy link
Contributor Author

IronRico commented Aug 9, 2020

Updates:
General Settings Info #34

I made some of the proposed changes on my own repo. Please see the screenshot below.

87886625-4f658080-c9e4-11ea-9bc2-638ebe3988c0


Please advise on what you think works or does not work. The wording is a bit of a place holder for the time being so feel free to advise on any concepts that need to be addressed.

Added an updated screenshot for the clear settings option
Updated the clear browser cache image
Updated page with added dialog in reference to the options shown.
Added new annotated screenshot for the clear browser cache option
Trimmed the content of the page to be more direct and updated the screenshot again.
Copy link
Contributor Author

@IronRico IronRico left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed the changes and found nothing to be out of range or in any form of extreme change.

Copy link
Contributor Author

@IronRico IronRico left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor changes were approved and implemented as requested.

@IronRico IronRico requested a review from jlblcc September 8, 2020 16:10
Copy link
Contributor

@hmaier-fws hmaier-fws left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should be referring to the "mdEditor storage cache" instead of the "browser cache" to avoid users confusing this with what they typically think of when clearing the cache (i.e. losing their browsing history or conversely, losing their mdEditor records if they clear their history)

reference/import-window/clear-cache.md Outdated Show resolved Hide resolved
IronRico and others added 2 commits September 9, 2020 12:45
I agree with the reduction on words here.  Good suggestion.

Co-authored-by: hamaier <[email protected]>
This was meant to be changed just as suggested.  Definitely agree.

Co-authored-by: hamaier <[email protected]>
Copy link
Contributor Author

@IronRico IronRico left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All changes were reviewed.

@hmaier-fws hmaier-fws dismissed jlblcc’s stale review December 17, 2020 21:35

Checked and approved. Josh unavailable

@hmaier-fws
Copy link
Contributor

Documentation updates

@hmaier-fws hmaier-fws closed this Dec 17, 2020
@hmaier-fws hmaier-fws reopened this Dec 17, 2020
@hmaier-fws hmaier-fws merged commit d2d1980 into adiwg:master Dec 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants