| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| | |
|
| | |
|
| |
|
|
| |
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
|
| |
|
|
| |
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
|
| | |
|
| |\
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Add support for inline videos in issue, MR and notes (on issue, commit, MR, and MR diff)
## What does this MR do?
It adds support for inline videos in issue, MR and notes (on issue, commit, MR, and MR diff). Most of the work was done by @hayesr in !3508 but a few improvements were still missing.
## Why was this MR needed?
To be able to play uploaded videos in GitLab!
## What are the relevant issue numbers?
Closes #4142.
## Screenshots
### Video players
-----
-----
## Does this MR meet the acceptance criteria?
- [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added
- [x] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md)
- Tests
- [x] Test `VideoLinkFilter`
- [x] Test in `spec/features/markdown_spec.rb`
- [x] Improve `spec/uploaders/file_uploader_spec.rb`
- [x] All builds are passing
- [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides)
- [x] Branch has no merge conflicts with `master` (if you do - rebase it please)
- [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits)
See merge request !5215
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
Handle videos in:
- MD preview in notes: commit, issue/MR, MR diff
- New notes in: commit, issue/MR, MR diff
- Persisted notes in: commit, issue/MR, MR diff
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
| |/ |
|
| |
|
|
|
|
| |
Add max-height to prevent images from displaying larger than the provided screen size.
Also fix a failing test and add a new one.
|
| | |
|
| | |
|
| |\
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Hide the Todo button in the collapsed issuable sidebar.
## What does this MR do?
Fixes a bug where the "Add Todo" button was displayed in the collapsed sidebar.
## Are there points in the code the reviewer needs to double check?
This shouldn't remove any other elements from the collapsed sidebar.
## What are the relevant issue numbers?
None
## Screenshots (if relevant)
Before:
After:
See merge request !4686
|
| | | |
|
| |\ \
| |/
|/|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Fixed issue with bold in issuable sidebar
## What does this MR do?
~~The sidebar contained both normal font weight & bold font weight, so this standardises them to bold.~~
After looking at the designs, i've updated the font weights & colors throughout the sidebar to correctly match them
## Screenshots (if relevant)
See merge request !4398
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | |
| |
| |
| |
| | |
This would happen when a todo already exists, the state of the button
wouldn't update after the ajax call
|
| | |
| |
| |
| |
| |
| | |
Added a button into the sidebar for issues & merge requests to allow users to manually create todo items
Closes #15045
|
| |/
|
|
|
| |
Added sidebar
Removed elements not present in design
|
| |
|
|
|
|
| |
In the sidebar when there is a due date, a link to remove due date becomes visible
Closes #17392
|
| |
|
|
| |
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
|
| |\ |
|
| | | |
|
| | |\
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Increased z-index of right sidebar
Fixes issue with active button overlapping it
Closes #15243
See merge request !3722
|
| | | |
| | |
| | |
| | |
| | |
| | | |
Fixes issue with active button overlapping it
Closes #15243
|
| |\ \ \
| |/ /
| | |
| | |
| | | |
# Conflicts:
# app/views/shared/issuable/_sidebar.html.haml
|
| | | |
| | |
| | |
| | |
| | |
| | | |
This will also work better for when the title makes the buttons drop down a new line
Closes #14228
|
| |/ /
| |
| |
| |
| | |
# Conflicts:
# app/assets/javascripts/labels_select.js.coffee
|
| |\ \
| |/
|/|
| |
| |
| |
| |
| |
| | |
Removed references to subscribe-button CSS class
These were being blocked by adblocks
Closes #15043
See merge request !3677
|
| | |
| |
| |
| |
| |
| | |
These were being blocked by adblocks
Closes #15043
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Fixed alignment on issuable new form
Fixes #13802
See merge request !3661
|
| | |/
| |
| |
| | |
Fixes #13802
|
| |/ |
|
| | |
|
| | |
|
| |
|
|
| |
Added nice animations too.
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
| |
Fixes violations of Shorthand.
Discussed in #14299.
[ci skip]
|
| | |
|
| |
|
|
| |
Fixed button hover color
|
| | |
|
| |
|
|
|
|
|
| |
Updated the spacing throughout the sidebar so that it fits better on smaller screens
Added more participants button
Closes #13353
|
| |\
| |
| |
| |
| |
| |
| |
| |
| | |
Edited timeago text on comments
Unified the 'edited text' to be the same in descriptions and comments
Closes #5538
See merge request !3063
|
| | |
| |
| |
| |
| |
| | |
Unified the 'edited text' to be the same in descriptions and comments
Closes #5538
|
| | |
| |
| |
| |
| |
| |
| |
| | |
Fixes violations of SpaceAfterPropertyColon.
Discussed in #14299.
[ci skip]
|
| |/
|
|
|
|
|
| |
Use lowercase letters and shorthand values for HEX colors where
possible.
Discussed in #14299.
|
| | |
|
