Skip to content

Removed some unwanted carriage returns in the Config.xml documentation page#1263

Merged
breautek merged 12 commits intoapache:masterfrom
DAnn2012:patch-4
Oct 1, 2022
Merged

Removed some unwanted carriage returns in the Config.xml documentation page#1263
breautek merged 12 commits intoapache:masterfrom
DAnn2012:patch-4

Conversation

@DAnn2012
Copy link
Contributor

Platforms affected

Config.xml page on cordova.apache.org https://cordova.apache.org/docs/en/11.x/config_ref/index.html

Motivation and Context

Removed some unwanted carriage returns (which could be the cause of some of the problems of not displaying the table form described here: #1232 ) and the obsolete reference to plugin cordova-plugin-whitelist in the example of the config.xml file

Description

Removed some unwanted carriage returns (which could be the cause of some of the problems of not displaying the table form described here: #1232 ) and the obsolete reference to plugin cordova-plugin-whitelist in the example of the config.xml file

@DAnn2012 DAnn2012 changed the title Update index.md Removed some unwanted carriage returns in the Config.xml documentation page Sep 27, 2022
@breautek
Copy link
Contributor

Generally speaking we should be updating the dev version. Updating the release versions can also be desirable sometimes, but if the issue is in a released version, chances are it's also in the dev version as well and the issue will reappear on our next release/snapshot if it's not corrected inside the dev folder.

I feel like most of these changes probably needs to be corrected in the dev folder as well, can you confirm?

@DAnn2012
Copy link
Contributor Author

DAnn2012 commented Sep 28, 2022

Hello @breautek

You're right. I made the changes in the dev folder as well.

Please also find a solution to the problem for which in the text of the description in the second column of the tables the HTML tags "ul" and "li" are shown:

Attribute "Min/Max Version" in "preference" (in the config.xml page)

config-xml

Attribute "name" in "engines and engine" (in the plugin.xml page)

plugin-xml

Thanks.

Update - Here it is written that it is a Kramdown problem: just-the-docs/just-the-docs#409

@breautek breautek merged commit 2fea3b6 into apache:master Oct 1, 2022
@breautek
Copy link
Contributor

breautek commented Oct 1, 2022

Thank you providing the much needed attention to the documentation :)

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.

2 participants