Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • G gitlabhq1
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 21
    • Issues 21
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 12
    • Merge requests 12
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Infrastructure Registry
  • Analytics
    • Analytics
    • CI/CD
    • Repository
    • Value stream
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • gpt
  • large_projects
  • gitlabhq1
  • Merge requests
  • !7530

Closed
Created Aug 18, 2014 by Administrator@rootOwner
  • Report abuse
Report abuse

Add ace-modes to javascript manifest, fixes #7513

  • Overview 23
  • Commits 2
  • Changes 4

Created by: Razer6

What does this MR do?

This PR adds most of the modes of ace editor to the javascript manifest. With this PR, there is syntax highlighting while editing a file. I've included all modes which file size is less than 120k.

Additionally it adds syntax highlighting, when creating a new snippet or file via web interface. The mode is determined on the current entered filename and tracks if the user changes it.

Are there points in the code the reviewer needs to double check?

Setting asset path of ace modes is removed, since this is now served directly through the rails asset pipeline.

Why was this MR needed?

Currently, when opening a file (e.g. a markdown), ace editor tries to load the mode file. Since it's not in the manifest, a 404 error is raised behind the scenes, and no syntax highlighting is available.

What are the relevant issue numbers / Feature requests?

This PR fixes

  • #7513 (closed)
  • #6850 (closed)
  • http://feedback.gitlab.com/forums/176466-general/suggestions/6300274-syntax-highlighting-in-editor-while-editing-and-cr

/cc @cirosantilli

Screenshots (If appropiate)

screenshot 2014-08-18 12 24 15

screenshot 2014-08-18 12 26 02

create_file

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: github/fork/Razer6/fix_ace_modes