summaryrefslogtreecommitdiff
path: root/app/views
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'project-title-dropdown' into 'master' Jacob Schatz2016-04-122-0/+10
|\ | | | | | | | | | | | | Project dropdown in header uses new dropdown See merge request !3339
| * Updated based on Ruby feedbackPhil Hughes2016-04-121-5/+6
| |
| * Updated testsPhil Hughes2016-04-121-1/+1
| |
| * Project dropdown in header uses new dropdownPhil Hughes2016-04-122-0/+9
| |
* | Merge branch 'member-access-note-row' into 'master' Jacob Schatz2016-04-121-6/+6
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixed issue with member access not being visible on notes This happened because the `note-actions`, which houses the access, was wrapped in a `can_edit` if statement ![Screen_Shot_2016-04-08_at_16.02.47](/uploads/40bffe9bb53c015f30ffa93bb018552b/Screen_Shot_2016-04-08_at_16.02.47.png) Fixes #15049 See merge request !3618
| * | Fixed issue with member access not being visible on notesmember-access-note-rowPhil Hughes2016-04-081-6/+6
| | | | | | | | | | | | Fixes #15049
* | | Merge branch 'fix_quick_submit_missing_in_edit_merge_request' into 'master' Jacob Schatz2016-04-121-1/+1
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | | | | Fix quick submit missing in edit merge request page closes #14953 ![fix](/uploads/5b80b37330929514c81299b4c8fe3efb/fix.gif) See merge request !3601
| * | fix quick submit missing in edit merge request pageArinde Eniola2016-04-071-1/+1
| | |
* | | Merge branch 'decouple-member-notification' into 'master' Douwe Maan2016-04-126-88/+85
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Decouple membership and notifications This allow you to have notification setting per project even if you are member of group. It also creates background for having notification settings in project you are not member of. - [x] Make it work - [x] Migrations - [x] CHANGELOG - [x] More tests - [x] API For #3359 After this merge request there is still some work to be done: * create migration that remove duplicates in notification settings table and create uniq index (8.8 probably) * remove notification_level field from Member model in 9.0 * make proper API for notification settings * use `MemberCreateService` instead of Member#after_create callback for creating notification settings (after #14709) * maybe more tests Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> See merge request !3421
| * | | Simplify Projects::NotificationSettingsControllerDouglas Barbosa Alexandre2016-04-121-1/+1
| | | |
| * | | Fix partial for update project notificationsDouglas Barbosa Alexandre2016-04-111-1/+1
| | | |
| * | | Merge branch 'master' into decouple-member-notificationDouglas Barbosa Alexandre2016-04-1147-257/+146
| |\ \ \
| * \ \ \ Merge branch 'master' into decouple-member-notificationDouglas Barbosa Alexandre2016-04-0840-228/+271
| |\ \ \ \ | | | |_|/ | | |/| |
| * | | | Change how notification settings in profile are rendered and updatedDmitriy Zaporozhets2016-03-305-58/+63
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | | Improve project notification settings explanationDmitriy Zaporozhets2016-03-301-1/+1
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | | Create NotificationSettings object only when user change value in dropdownDmitriy Zaporozhets2016-03-301-3/+2
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | | Introduce NotificationSetting to user interfaceDmitriy Zaporozhets2016-03-303-39/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * visiting project will create notification setting if missing * change notification setting per project even without membership * use notification settings instead of membership on profile page Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | | | Merge branch 'commit-message-prefilled' into 'master' Jacob Schatz2016-04-121-1/+1
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prefills commit message in edit file See #14488 ![Screen_Shot_2016-04-06_at_10.18.06](/uploads/2ace2a21a6ead8d83c79ef9ce5dd34cc/Screen_Shot_2016-04-06_at_10.18.06.png) See merge request !3564
| * | | | | Added back text used on merge commit messagecommit-message-prefilledPhil Hughes2016-04-111-1/+1
| | | | | |
| * | | | | Prefills commit message in edit filePhil Hughes2016-04-111-1/+1
| | |_|/ / | |/| | | | | | | | | | | | | See #14488
* | | | | Merge branch 'build-notification-fixes' into 'master' Jacob Schatz2016-04-121-7/+9
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Build notification null check Cancels build notification interval on page change See merge request !3540
| * | | | | Fixed failing testsbuild-notification-fixesPhil Hughes2016-04-111-1/+1
| | | | | |
| * | | | | Preparing build textPhil Hughes2016-04-111-1/+8
| | | | | |
| * | | | | Build notification null checkPhil Hughes2016-04-111-6/+1
| |/ / / / | | | | | | | | | | | | | | | Cancels build notification interval on page change
* | | | | hide help block when user is creating a new project inside a groupArinde Eniola2016-04-121-1/+1
|/ / / /
* | | | Merge branch 'rs-form_errors' into 'master' Dmitriy Zaporozhets2016-04-1131-153/+41
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Standardize the way we check for and display form errors - Some views had a "Close" button. We've removed this, because we don't want users accidentally hiding the validation errors and not knowing what needs to be fixed. - Some views used `li`, some used `p`, some used `span`. We've standardized on `li`. - Some views only showed the first error. We've standardized on showing all of them. - Some views added an `#error_explanation` div, which we've made standard. See merge request !3531
| * | | | Standardize the way we check for and display form errorsrs-form_errorsRobert Speicher2016-04-0431-153/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Some views had a "Close" button. We've removed this, because we don't want users accidentally hiding the validation errors and not knowing what needs to be fixed. - Some views used `li`, some used `p`, some used `span`. We've standardized on `li`. - Some views only showed the first error. We've standardized on showing all of them. - Some views added an `#error_explanation` div, which we've made standard.
* | | | | Merge branch 'mr-ui-update' into 'master' Jacob Schatz2016-04-119-77/+79
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updated UI for new merge request Closes #2540 ![Screen_Shot_2016-03-29_at_12.53.18](/uploads/426e603675171f0dc4e0af83c7762eba/Screen_Shot_2016-03-29_at_12.53.18.png) ![Screen_Shot_2016-03-29_at_12.53.13](/uploads/645ad7955acfb9f7693245fcc048ee49/Screen_Shot_2016-03-29_at_12.53.13.png) See merge request !3228
| * | | | | Addressed feedbackPhil Hughes2016-04-112-8/+11
| | | | | | | | | | | | | | | | | | | | | | | | Removed important from css
| * | | | | Fixed buildsPhil Hughes2016-04-062-12/+5
| | | | | |
| * | | | | Fixed target project updatePhil Hughes2016-04-063-12/+8
| | | | | |
| * | | | | Checkmarks in dropdowns for already selected valuesPhil Hughes2016-04-061-7/+7
| | | | | |
| * | | | | Use new dropdowns for MR comparePhil Hughes2016-04-061-8/+47
| | | | | |
| * | | | | Updated UI for new merge requestPhil Hughes2016-04-066-55/+26
| | | | | | | | | | | | | | | | | | | | | | | | Closes #2540
* | | | | | Merge branch 'prevent_users_from_upvoting_downvoting_emoji' into 'master' Jacob Schatz2016-04-111-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | prevent users from being able to both upvote and downvote See merge request !3495
| * | | | | | prevent users from being able to both upvote and downvoteArinde Eniola2016-04-091-1/+1
| | | | | | |
* | | | | | | Merge branch 'rs-markdown-preview-tabindex' into 'master' Jacob Schatz2016-04-101-3/+3
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove tab stop from "Write", "Preview", "Go full screen" links Closes #15089 See merge request !3635
| * | | | | | | Remove tab stop from "Write", "Preview", "Go full screen" linksrs-markdown-preview-tabindexRobert Speicher2016-04-091-3/+3
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | Closes #15089
* | | | | | | Merge branch 'redesign_labels_page' into 'master' Jacob Schatz2016-04-102-8/+12
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Redesign labels page closes #14227 ## Desktop ![Screen_Shot_2016-04-05_at_7.19.37_PM](/uploads/a52b764565b6e2739fb6932178f8bde5/Screen_Shot_2016-04-05_at_7.19.37_PM.png) ## Mobile ![Screen_Shot_2016-04-04_at_6.40.57_PM](/uploads/9804b074126fcdb6755918e9f4c472a7/Screen_Shot_2016-04-04_at_6.40.57_PM.png) See merge request !3507
| * | | | | | fix the failing tests and some changesArinde Eniola2016-04-061-4/+4
| | | | | | |
| * | | | | | finish up the design and add info to the changelogArinde Eniola2016-04-061-4/+4
| | | | | | |
| * | | | | | change the subscribe, delete and edit buttons to iconsArinde Eniola2016-04-062-8/+12
| | | | | | |
* | | | | | | Merge branch '14400-diff-redesign' into 'master' Jacob Schatz2016-04-095-18/+9
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Diff redesign Fixes #14400 Fixes #13500 ![diffs_redesign](/uploads/16da9557e497114d886ef5ece2f306b3/diffs_redesign.gif) See merge request !3476
| * | | | | | | Remove comment count & icon from side by side view14400-diff-redesignAnnabel Dunstone2016-04-071-6/+0
| | | | | | | |
| * | | | | | | Diff touch upsAnnabel Dunstone2016-04-071-1/+1
| | | | | | | |
| * | | | | | | Remove comment count & iconAnnabel Dunstone2016-04-071-3/+0
| | | | | | | |
| * | | | | | | Diff design updatesAnnabel Dunstone2016-04-071-5/+5
| | | | | | | |
| * | | | | | | Update diff colors and iconsAnnabel Dunstone2016-04-072-3/+3
| | |_|_|_|/ / | |/| | | | |
* | | | | | | Merge branch 'efficient-emoji-loading' into 'master' Robert Speicher2016-04-091-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Only load emoji spritesheet when necessary On a normal issue where the only award emoji displayed by default are the thumbs-up and thumbs-down emoji, this decreases the page load by 670KB or 250KB depending on pixel density. Resolves #14334. See merge request !3449
| * | | | | | | Add "sprite" parameter to emoji_icon helperconnorshea2016-04-081-1/+1
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The emoji_icon helper used to display the award emoji in Issue and Merge Request views. By default the spritesheet is used, but passing `sprite: false` to the `emoji_icon` helper makes the emoji render as separate images. For award emoji displayed by default (e.g. thumbs-up, thumbs-down, and any that have been awarded to the issue/MR) the independent images are used. Only when the full emoji menu is opened does the full spritesheet load. On a normal issue this change decreases the page load by 670KB or 250KB depending on pixel density. Resolves #14334.