Skip to content
This repository has been archived by the owner on Aug 23, 2024. It is now read-only.

Summaries/comments not indented properly on deploy #189

Closed
THDigi opened this issue Apr 18, 2020 · 6 comments
Closed

Summaries/comments not indented properly on deploy #189

THDigi opened this issue Apr 18, 2020 · 6 comments
Labels

Comments

@THDigi
Copy link
Collaborator

THDigi commented Apr 18, 2020

Result: https://gist.github.com/THDigi/a3eb524e0bd971bd3fddca55cc5e7515/ae0751000a6371ec8df188eda49bb73dfbc3655a#file-simpletimersm_example_mergedforpb-cs-L54

The SimpleTimerSM is defined in a referenced mixin project and also has:
// <mdk sortorder="999" />
at its start.

@Gwindalmir
Copy link

Your example looks fine, however I am hitting this issue as well.
I have several "utility classes" in my project, and when deployed, the XML Documentation tags aren't at the same indent level.

Here's a screenshot in-game after pasting the contents from the deploy notification:
image

@THDigi
Copy link
Collaborator Author

THDigi commented Dec 5, 2020

Oh I manually fixed indentation in that gist, I forgot I posted it here =)

You can see the fixes in revisions, it's the same as your screenshot yeah.

@tgkgate
Copy link

tgkgate commented Sep 4, 2021

Can confirm this is still an issue 2021/09/04 with xmldoc comments

/// <summary>
	  /// Class containing extension methods for the following classes<br/>
	  /// <br/>
	  /// GridTerminalSystem
	  /// </summary>
public static class Extensions

@tgkgate
Copy link

tgkgate commented May 2, 2023

Can reconfirm as of 2023/05/02 this bug is alive and well.

/// <summary>
        /// Sets the name of the remote control to orient on (e.g. "remote control 1")<br />
        /// Set to "" to use the first remote control found
        /// </summary>
        /// <remarks>
        ///	Default: Empty String
        /// </remarks>
private const string REMOTE_CONTROL_NAME = "";

@malware-dev
Copy link
Owner

I'm afraid this does not have a high priority.

@malware-dev
Copy link
Owner

Closing because this site is being archived due to complete replacement at https://github.com/malforge/mdk2. If issue persist, please create new issue there.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants