Skip to content
This repository was archived by the owner on Nov 19, 2024. It is now read-only.

Feedback on page: /guides/v2.1/javascript-dev-guide/widgets/widget_modal.html #2014

Closed
2 tasks done
korostii opened this issue May 31, 2018 · 4 comments
Closed
2 tasks done
Assignees
Labels
2.1.x Editorial Typo and grammar fixes or minor rewrites to correct inaccuracies Technical Updates to the code or processes that alter the technical content of the doc

Comments

@korostii
Copy link
Contributor

This issue is a:

  • Topic clarification request
  • New DevDocs feature request

Description:

The documentation page corresponding to the modal widget seems to omit the fact that the clickableOverlay parameter isn't working as expected in any of the released versions and will only be fixed in the future 2.2.5 version.

Steps to reproduce

  1. Observe https://devdocs.magento.com/guides/v2.1/javascript-dev-guide/widgets/widget_modal.html
  2. Observe Modal UI: clickableOverlay option doesn't work magento2#7399
  3. Note that the devdocs page has no reference to that issue nor any mention of anything of the sort.

Expected result:

I would expect the page to contain a mention of that issue, or a reference to it, or a comment section where a community developer could post such a notice for other fellow developers to see.

Possible solutions:

a) add a short mention of that issue to the corresponding documentation issue
b) add a reference to the GitHub issue magento/magento2#7399
c) create a comment section where a community developer could post such a notice for other fellow developers to see.

@bdenham
Copy link
Contributor

bdenham commented Jun 1, 2018

Hey @korostii, good catch on this. Would you be willing to submit a pull request with the updates you're recommending?

If so, here are some basic instructions on how to contribute to devdocs on our DevDocs Contributions page.

If not, let me know and I'll submit a PR for you.

@bdenham bdenham added Editorial Typo and grammar fixes or minor rewrites to correct inaccuracies 2.1.x Technical Updates to the code or processes that alter the technical content of the doc Waiting for Response Waiting for response from internal/external parties labels Jun 1, 2018
@korostii
Copy link
Contributor Author

korostii commented Jun 5, 2018

Hi @bdenham, thanks for the timely response.

I am pretty sure there are Magento's internal policies or legalese restrictions for acknowledging issues but I don't know the specifics.
For reference, there is a lengthy discussion on that topic in #1268 (long story short, was requesting a similar edit on a different page and got turned down).

That's one of the reasons why I am a little hesitant to submit any significant changes (or even invest time and effort into them, at all).
Don't get me wrong, I would be very glad if you could do such an edit, by all means, feel free to do so as you like.

Just please make sure it's okay to do so, I wouldn't want to get you in trouble =)
Keep up the good work & have a nice day!

@bdenham
Copy link
Contributor

bdenham commented Jun 5, 2018

@korostii, thanks for the additional info! Interesting. I'll research this issue more to get a better understanding. Thanks, again.

@bdenham bdenham added groomed and removed Waiting for Response Waiting for response from internal/external parties labels Jun 5, 2018
@dshevtsov
Copy link
Collaborator

@korostii, according to magento/magento2#7399 the issue has been fixed. If you still observe any documentation issues in 2.2 or 2.3, please let us know.

Note: This issue/pull request applies to 2.1 only. We are no longer accepting pull requests for 2.1 DevDoc updates, because this version reached end of life (EOL) on June 2019. Magento no longer applies security patches, quality fixes, or documentation updates to 2.1.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
2.1.x Editorial Typo and grammar fixes or minor rewrites to correct inaccuracies Technical Updates to the code or processes that alter the technical content of the doc
Projects
None yet
Development

No branches or pull requests

3 participants