1. 09.06.2016 14 коммитов
    • Z.J. van de Weg's avatar
      Fix migration when converting award notes
      · 216436c1
      Z.J. van de Weg создал
      216436c1
    • Rémy Coutable's avatar
      Merge branch '18019-fix-wiki-linking' into 'master'
      · 3d72cb89
      Rémy Coutable создал
      Fix wiki linking behavior for markdown wiki pages
      
      Related to #18019 
      
      - As per the documentation in !4372 
      
      ## TODO
      - [ ] !4432 Have wiki linking behave as per the documentation
          - [x] Move `WikiLinkFilter` specs to the pipeline level
          - [x] Verify current behavior on wiki `show` page
          - [x] Fix current behavior on wiki `show` page
          - [x] Verify current behaviour on wiki preview
          - [x] Fix current behaviour on wiki preview
              - [x] Rewrite all links and get preview links working
              - [x] Make sure all links are on-par with the wiki `show` page
              - [x] TDD `WikiLinkFilter` and get it working
              - [x] Hook `WikiLinkFilter` up
          - [x] Fix tests 
              - [x] Fix `markdown_spec`
              - [x] Fix `wiki` spinach feature
              - [x] Wait for [build](https://gitlab.com/gitlab-org/gitlab-ce/commit/4f50dd2/builds) to pass
              - [x] Make sure all wiki-related pages are working as expected (histor...
      3d72cb89
    • Timothy Andrew's avatar
      Add #18019 to the CHANGELOG.
      · 19b91e74
      Timothy Andrew создал
      19b91e74
    • Timothy Andrew's avatar
      Hook up the updated `WikiLinkFilter` to the wiki controllers.
      · e6b1d166
      Timothy Andrew создал
      - Need to pass in a `page_slug` to the filter, so it can rewrite based
        on the current page (all links are rewritten to the level of the app root).
      - The earlier `markdown_preview` endpoint was at the level of the wiki.
        We need to know the current page (for rewriting, as above), so this
        commit moves the endpoint to the level of a wiki page.
      - Fix all tests
      e6b1d166
    • Timothy Andrew's avatar
      Implement the correct linking behaviour in `WikiLinkFilter`.
      · 8e71c19a
      Timothy Andrew создал
      Original Comments
      =================
      
      - Linking behaves as per rules documented here:
        https://gitlab.com/gitlab-org/gitlab-ce/blob/16568-document-wiki-linking-behavior/doc/markdown/wiki.md
      - All links (to other wiki pages) are rewritten to be at the level of
        the app root. We can't use links relative to the current
        page ('./foo', 'foo', '../foo'), because they won't work in the
        markdown preview, where the current page is suffixed with `/edit`
      - Move existing `WikiLinkFilter` specs to `WikiPipeline` spec. It makes
        sense to run these tests on the combined output of the pipeline,
        rather than a single filter, since we can catch issues with
        conflicting filters.
      - Add more tests to cover the new linking
      
      @rymai's Review
      ===============
      
      - Classes nested under `WikiLinkFilter` should declare `WikiLinkFilter`'s
        inherit, so nothing changes if the nested class is loaded first.
      - Add a blank line after a guard clause
      - Use keyword arguments for the `Rewriter` constructor
      - Invert a condition - use `if` instead of `unless`
      - Inline a `let` in `WikiPipeline` spec - it was only used in a single place
      - Change out of date spec names
      - Add a comment for every rewrite rule in `Rewriter`
      8e71c19a
    • Jacob Schatz's avatar
      Merge branch 'fix-award-emoji-hidden' into 'master'
      · 13fd88fa
      Jacob Schatz создал
      Shows award emoji for comments to all users who are logged in.
      
      ## What does this MR do?
      Fixes the award emoji picker not showing up for users who weren't able to edit another user's comment.
      
      ## Screenshots (if relevant)
      Before:
      
      ![Screen_Shot_2016-06-08_at_5.18.04_PM](/uploads/a8b40ba319481f53d55de055d84649b9/Screen_Shot_2016-06-08_at_5.18.04_PM.png)
      
      After (logged in):
      ![Screen_Shot_2016-06-08_at_5.16.16_PM](/uploads/535edcca6fbdd8834beda39399f85384/Screen_Shot_2016-06-08_at_5.16.16_PM.png)
      
      After (logged out):
      ![Screen_Shot_2016-06-08_at_5.15.08_PM](/uploads/7f0dff751714c7d20f78b56893cacd59/Screen_Shot_2016-06-08_at_5.15.08_PM.png)
      
      cc: @fatihacet 
      
      See merge request !4547
      13fd88fa
    • Jacob Schatz's avatar
      Merge branch 'mention-loading-indicator' into 'master'
      · f5818baf
      Jacob Schatz создал
      Show loading indicator for autocomplete
      
      ![autocomplete](/uploads/397df34689aeb5f4cbca99aea209c8cf/autocomplete.gif)
      
      Closes #15435
      
      See merge request !4182
      f5818baf
    • Jacob Schatz's avatar
      Merge branch '17284-Edit-form-does-not-cancel-correctly' into 'master'
      · e382f130
      Jacob Schatz создал
      #17284 Edit form does not cancel correctly
      
      ## What does this MR do?
      
      Replaces the markdown textarea with the currently saved note text when the note edit is cancelled.
      
      ## Are there points in the code the reviewer needs to double check?
      
      No
      
      ## Why was this MR needed?
      
      Allows the user to revert back to the original note text, as expected, using cancel.
      
      ## What are the relevant issue numbers?
      
      Fixes #17284.
      
      ## Screenshots
      
      ![17284](/uploads/c3598de6f8e70d52275ef25907d92d42/17284.gif)
      
      See merge request !4175
      e382f130
    • Jacob Schatz's avatar
      Merge branch 'nav-scroll-bug' into 'master'
      · cc54b74a
      Jacob Schatz создал
      Side nav scrolls if content height taller than screen
      
      ## What does this MR do?
      Header logo is now fixed at top; put back scrolling functionality in side nav
      
      ## Screenshots (if relevant)
      <img src="/uploads/e1d1b60c296c39c61d7fe7da22b6c52d/scroll.gif" width="600px">
      
      See merge request !4513
      cc54b74a
    • Robert Speicher's avatar
      Merge branch 'git-http-no-authenticity-token' into 'master'
      · 5d66e0e0
      Robert Speicher создал
      Disable CSRF protection when serving Git HTTP clients
      
      Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/18331
      
      See merge request !4538
      5d66e0e0
    • Connor Shea's avatar
      bdc995bf
    • Jacob Schatz's avatar
      Merge branch '17240-svg' into 'master'
      · 0001a0cf
      Jacob Schatz создал
      Add new icons in left sidebar and layout nav
      
      Closes #17609    
      Closes #17240    
      Closes #17321    
      
      ![Screen_Shot_2016-06-02_at_8.47.38_AM](/uploads/f254fe81cba65cbdeb8e5720f0b2fc19/Screen_Shot_2016-06-02_at_8.47.38_AM.png)
      ![Screen_Shot_2016-06-02_at_8.53.33_AM](/uploads/2c27284f0a4916fe447794d9b663c4df/Screen_Shot_2016-06-02_at_8.53.33_AM.png)
      
      @skyruler I couldn't find the icons for `Projects`, `Merge Requests`, `Snippets`, or `Help` in the design repo. Do you have them available somewhere?
      
      See merge request !4268
      0001a0cf
    • Jacob Schatz's avatar
      Merge branch '18333-layout-bug' into 'master'
      · 8578f401
      Jacob Schatz создал
      Use container helper instead of hard coded container
      
      ## What does this MR do?
      Add `container_class` helper class to fix layout bug
      
      ## What are the relevant issue numbers?
      #18333 
      ## Screenshots (if relevant)
      Fixed:    
      ![Screen_Shot_2016-06-08_at_2.03.38_PM](/uploads/6e7575d711efb8eaf6ba641907b2d607/Screen_Shot_2016-06-08_at_2.03.38_PM.png)
      Fluid:     
      ![Screen_Shot_2016-06-08_at_2.03.56_PM](/uploads/33d3f73781ca9ed129f853abfe9feb4b/Screen_Shot_2016-06-08_at_2.03.56_PM.png)
      
      See merge request !4543
      8578f401
    • Annabel Dunstone's avatar
      Fix activity SVG
      · a280116a
      Annabel Dunstone создал
      a280116a
  2. 08.06.2016 26 коммитов