summaryrefslogtreecommitdiff
path: root/lib/banzai
Commit message (Collapse)AuthorAgeFilesLines
* Remove N+1 queries when checking nodes visible to userfix-n-plus-one-project-featuresSean McGivern2017-05-021-1/+2
| | | | | | | N in this case is low, as it's the number of distinct projects referenced from MRs, rather than the number of MRs referenced (issues use their own optimization). Still, on issues or MRs which are often referenced from MRs, it will save a few queries.
* Merge branch 'blackst0ne/gitlab-ce-fix_emoji_parser'Sean McGivern2017-04-271-1/+4
|\
| * Fix rendering emoji inside a stringblackst0ne2017-04-251-1/+4
| |
* | issues_for_nodes => issuables_for_nodes31280-skip-issueables-without-projectLin Jen-Shin2017-04-261-3/+3
| |
* | Follow feedback on the reviewLin Jen-Shin2017-04-261-4/+4
| |
* | Skip issuable without a project in IssuableExtractor#extractLin Jen-Shin2017-04-251-1/+5
| | | | | | | | Closes #31280
* | Start versioning cached markdown fieldsNick Thomas2017-04-211-14/+7
| |
* | Remove a use of module_functionNick Thomas2017-04-211-12/+10
| |
* | Merge branch 'issuable-state-custom-links' into 'master' Sean McGivern2017-04-201-2/+2
|\ \ | | | | | | | | | | | | | | | | | | Do not append issuable state to links with custom anchor Closes #30916 See merge request !10782
| * | Do not append issuable state to links with custom anchorissuable-state-custom-linksAdam Niedzielski2017-04-191-2/+2
| | | | | | | | | | | | | | | Closes #30916 Change the presentation from [closed] to (closed)
* | | Merge branch 'plantuml-filter-after-highlight' into 'master' Sean McGivern2017-04-201-4/+4
|\ \ \ | |/ / |/| | | | | | | | | | | | | | Fix PlantUML integration in GFM Closes #30557 See merge request !10651
| * | Fix PlantUML integration in GFMplantuml-filter-after-highlightAdam Niedzielski2017-04-191-4/+4
| | | | | | | | | | | | | | | Fixes #30557 Use "lang" attribute instead of "class" in PlantumlFilter.
* | | Display issuable state only in notes and issuable descriptionAdam Niedzielski2017-04-191-0/+2
|/ / | | | | | | Closes #30964
* | Fix regression in rendering Markdown references that do not existsh-fix-base-parserStan Hu2017-04-151-1/+2
| | | | | | | | Closes #30972
* | Fix appending state to issuable referencesfix-appending-state-to-issuable-referencesAdam Niedzielski2017-04-131-2/+2
|/ | | | | | | Closes #30874. Ignore nodes with no children. Append directly to the node instead of the last child of the node to avoid inheriting formatting from the last child
* Reuse "merge_requests_for_nodes" in "MergeRequestParser"merge-request-parser-merge-requests-for-nodesAdam Niedzielski2017-04-101-11/+30
|
* Add indication for closed or merged issuables in GFMadam-finish-5993-closed-issuableAdam Buckland2017-04-079-40/+124
| | | | | | | | | | | | | | | Example: for issues that are closed, the links will now show '[closed]' following the issue number. This is done as post-process after the markdown has been loaded from the cache as the status of the issue may change between the cache being populated and the content being displayed. In order to avoid N+1 queries problem when rendering notes ObjectRenderer populates the cache of referenced issuables for all notes at once, before the post processing phase. As a part of this change, the Banzai BaseParser#grouped_objects_for_nodes method has been refactored to return a Hash utilising the node itself as the key, since this was a common pattern of usage for this method.
* Merge branch '30125-markdown-security'Robert Speicher2017-04-055-25/+16
| | | | | | Remove class from SanitizationFilter whitelist See merge request !2079
* Merge branch 'make_user_mentions_case_insensitive' into 'master' Sean McGivern2017-03-291-2/+2
|\ | | | | | | | | | | | | Make user mentions case-insensitive Closes #24493 See merge request !10285
| * Make user mentions case-insensitiveblackst0ne2017-03-291-2/+2
| |
* | Get rid of a third N+1 query problemRémy Coutable2017-03-281-1/+3
| | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | Get rid of second N+1 query problemRémy Coutable2017-03-281-0/+4
| | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | Get rid of a first N+1 query problemRémy Coutable2017-03-281-2/+21
|/ | | | | | | Use the same approach in Banzai::Filter::MergeRequestReferenceFilter as we do with issues. Signed-off-by: Rémy Coutable <remy@rymai.me>
* Merge branch 'fix-links-target-blank' into 'security'Jacob Schatz2017-03-202-2/+3
| | | | | | Adds rel="noopener noreferrer" to all links with target="_blank" See merge request !2071
* Use Enumerable#index_by where possibledm-use-index-byDouwe Maan2017-03-163-13/+3
|
* remove all references to the .es6 in our config files and documentationremove-es6-extension-configurationMike Greiling2017-03-151-2/+2
|
* Copy code as GFM from diffs, blobs and GFM code blocksDouwe Maan2017-03-141-12/+1
|
* use a magic default :global symbol instead of nilhttp://jneen.net/2017-03-091-1/+1
| | | | to make sure we mean the global permissions
* Merge branch '26371-native-emojis-v3' into 'master' Filipa Lacerda2017-03-071-53/+11
|\ | | | | | | | | | | | | Add emoji images - Base Native Unicode Emojis Closes #26371 See merge request !9569
| * Use native unicode emojisEric Eastwood2017-03-061-53/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - gl_emoji for falling back to image/css-sprite when the browser doesn't support an emoji - Markdown rendering (Banzai filter) - Autocomplete - Award emoji menu - Perceived perf - Immediate response because we now build client-side - Update `digests.json` generation in gemojione rake task to be more useful and include `unicodeVersion` MR: !9437 See issues - #26371 - #27250 - #22474
* | SanitizationFilter allows html5 details and summary (Issue #21605)James2017-03-061-0/+4
|/ | | | Also adds details/summary tags to Copy-as-GFM
* Use full group name in GFM group reference titledm-group-reference-full-nameDouwe Maan2017-03-011-1/+1
|
* Merge branch '20890-comment-image-inline' into 'master' 28711-cherry-picking-ui-regressions-8-17-0Robert Speicher2017-02-271-8/+1
|\ | | | | | | | | | | | | Fix inline comment images by removing wrapper #20890 Closes #20890 See merge request !5804
| * Fix inline comment images by removing wrapper #20890Jared Deckard2016-08-131-8/+1
| |
* | Revert "Enable Style/DotPosition"Douwe Maan2017-02-232-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit e00fb2bdc2090e9cabeb1eb35a2672a882cc96e9. # Conflicts: # .rubocop.yml # .rubocop_todo.yml # lib/gitlab/ci/config/entry/global.rb # lib/gitlab/ci/config/entry/jobs.rb # spec/lib/gitlab/ci/config/entry/factory_spec.rb # spec/lib/gitlab/ci/config/entry/global_spec.rb # spec/lib/gitlab/ci/config/entry/job_spec.rb # spec/lib/gitlab/ci/status/build/factory_spec.rb # spec/lib/gitlab/incoming_email_spec.rb
* | Revert "Prefer leading style for Style/DotPosition"Douwe Maan2017-02-234-10/+10
| | | | | | | | This reverts commit cb10b725c8929b8b4460f89c9d96c773af39ba6b.
* | Revert "Enable Style/BarePercentLiterals"Douwe Maan2017-02-233-6/+6
| | | | | | | | This reverts commit 96bef54154e669f9a3e92c3a4bc76c0be3a52e48.
* | Prefer leading style for Style/DotPositionDouwe Maan2017-02-234-10/+10
| |
* | Fix code for copsDouwe Maan2017-02-233-15/+18
| |
* | Enable Style/ConditionalAssignmentDouwe Maan2017-02-233-9/+9
| |
* | Enable Style/BarePercentLiteralsDouwe Maan2017-02-233-6/+6
| |
* | Enable Style/MutableConstantDouwe Maan2017-02-231-1/+1
| |
* | Enable Style/DotPositionDouwe Maan2017-02-232-4/+4
| |
* | No more and/orDouwe Maan2017-02-211-1/+1
| |
* | Only yield valid references in ReferenceFilter.references_inonly-yield-valid-reference-matchesDouwe Maan2017-02-161-1/+6
| |
* | Add GFM support to nested groupsdz-nested-groups-gfm-altDmitriy Zaporozhets2017-02-132-4/+4
| | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | Create directly_addressed Todos when mentioned in beginning of a lineErshad Kunnakkadan2017-02-084-7/+67
| |
* | Merge branch 'markdown-plantuml' into 'master' Sean McGivern2017-02-032-0/+40
|\ \ | | | | | | | | | | | | | | | | | | PlantUML support for Markdown Closes #4048 See merge request !8588
| * | PlantUML support for MarkdownHoracio Sanson2017-02-032-0/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow rendering of PlantUML diagrams in Markdown documents using fenced blocks: ```plantuml Bob -> Sara : Hello Sara -> Bob : Go away ``` Closes: #4048
* | | replace `find_with_namespace` with `find_by_full_path`Adam Pahlevi2017-02-031-1/+1
|/ / | | | | | | add complete changelog for !8949