Skip to content
Snippets Groups Projects

Encyclopedia Slave Leadership

Merged Blank_Alt requested to merge Blank_Alt/fc-pregmod:encyclopediaSlaveLeaders into pregmod-master
  • Said section of !10927 (merged), converted over to the desired format. (src/gui/Encyclopedia/encyclopedia.js is added but not implemented, mainly to help ease future conversions.)
  • showArticleInDialog is only used within src/gui/Encyclopedia/encyclopediaDialog.js, so it seems redundant to expose it.
  • Adds a minor tweak to makeLinkDOM which should help improve readability for links where text and article are identical.
Edited by Blank_Alt

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Arkerthan
  • Blank_Alt added 1 commit

    added 1 commit

    • 108aaf05 - Space to Tabs, implement suggestions and minor encylopedia formatting.

    Compare with previous version

  • Blank_Alt resolved all threads

    resolved all threads

  • Blank_Alt added 1 commit

    added 1 commit

    Compare with previous version

  • Blank_Alt added 1 commit

    added 1 commit

    Compare with previous version

  • Blank_Alt resolved all threads

    resolved all threads

    • Did the related links get moved deeper into the file or something? Hard to track things like that down at times.

    • Did the related links get moved deeper into the file or something? Hard to track things like that down at times.

      Related links for slaveLeadership has been moved from the dedicated twine file (with manual spaced setup) to the end of the relevant JS file that also contains the applicable articales (now using automatic spacing due to App.UI.DOM.generateLinksStrip).

      Edited by Blank_Alt
    • All tested?

    • Yes, no errors to report.

    • Please register or sign in to reply
  • merged

  • Pregmodder mentioned in commit bc94d960

    mentioned in commit bc94d960

  • Blank_Alt mentioned in merge request !10982 (merged)

    mentioned in merge request !10982 (merged)

  • Blank_Alt mentioned in merge request !10981 (merged)

    mentioned in merge request !10981 (merged)

  • Blank_Alt mentioned in merge request !10984 (merged)

    mentioned in merge request !10984 (merged)

  • mentioned in issue #4234 (closed)

  • Blank_Alt mentioned in merge request !11026 (merged)

    mentioned in merge request !11026 (merged)

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading