Merge branch 'master' into SKrastev/grid-disabled-pin-row

Author: ChronosSF

projects/igniteui-angular/src/lib/core/styles/components/splitter/_splitter-theme.scss

@@ -93,7 +93,6 @@
         align-items: center;
         background: $splitter-color;
         border: 1px solid $splitter-color;
-        cursor: row-resize;
         z-index: 99;
         opacity: .68;
         transition: opacity .15s $ease-out-quad !important;
@@ -123,7 +122,6 @@
     %igx-splitter-bar--vertical {
         flex-direction: column;
         height: 100%;
-        cursor: col-resize;
 
         &::before {
             @extend %handle-area--vertical;

projects/igniteui-angular/src/lib/splitter/splitter-bar/splitter-bar.component.html

@@ -1,5 +1,6 @@
 <div class="igx-splitter-bar"
     [class.igx-splitter-bar--vertical]='type === 0'
+    [style.cursor]='cursor'
     igxDrag
     [ghost]="false"
     [dragDirection]='dragDir'

projects/igniteui-angular/src/lib/splitter/splitter-bar/splitter-bar.component.ts

@@ -7,10 +7,8 @@ import {DragDirection, IDragMoveEventArgs, IDragStartEventArgs} from '../../dire
 export const SPLITTER_INTERACTION_KEYS = new Set('right down left up arrowright arrowdown arrowleft arrowup'.split(' '));
 
 /**
- * Provides reference to `SplitBarComponent` component.
- * Represents the draggable gripper that visually separates panes and allows for changing their sizes.
- * @export
- * @class SplitBarComponent
+ * @hidden @internal
+ * Represents the draggable bar that visually separates panes and allows for changing their sizes.
  */
 @Component({
     selector: 'igx-splitter-bar',
@@ -24,15 +22,13 @@ export class IgxSplitBarComponent {
     public cssClass = 'igx-splitter-bar-host';
 
     /**
-     * Sets/gets `IgxSplitBarComponent` orientation.
-     * @type SplitterType
+     * Gets/Sets the orientation.
      */
     @Input()
-    public type: SplitterType = SplitterType.Vertical;
+    public type: SplitterType = SplitterType.Horizontal;
 
     /**
-     * Sets/gets `IgxSplitBarComponent` element order.
-     * @type SplitterType
+     * Sets/gets the element order.
      */
     @HostBinding('style.order')
     @Input()
@@ -43,7 +39,29 @@ export class IgxSplitBarComponent {
      * @internal
      */
     @HostBinding('attr.tabindex')
-    public tabindex = 0;
+    public get tabindex() {
+        return this.resizeDisallowed ? null : 0;
+    }
+
+    /**
+     * @hidden
+     * @internal
+     */
+    @HostBinding('attr.aria-orientation')
+    public get orientation() {
+        return this.type === SplitterType.Horizontal ? 'horizontal' : 'vertical';
+    }
+
+    /**
+     * @hidden
+     * @internal
+     */
+    public get cursor() {
+        if (this.resizeDisallowed) {
+            return '';
+        }
+        return this.type === SplitterType.Horizontal ? 'col-resize' : 'row-resize';
+    }
 
     /**
      * Sets/gets the `SplitPaneComponent` associated with the current `SplitBarComponent`.
@@ -60,35 +78,26 @@ export class IgxSplitBarComponent {
 
     /**
      * An event that is emitted whenever we start dragging the current `SplitBarComponent`.
-     * @memberof SplitBarComponent
      */
     @Output()
     public moveStart = new EventEmitter<IgxSplitterPaneComponent>();
 
     /**
      * An event that is emitted while we are dragging the current `SplitBarComponent`.
-     * @memberof SplitBarComponent
      */
     @Output()
     public moving = new EventEmitter<number>();
 
-    /**
-     * An event that is emitted when collapsing the pane
-     */
-    @Output()
-    public togglePane = new EventEmitter<IgxSplitterPaneComponent>();
     /**
      * A temporary holder for the pointer coordinates.
-     * @private
-     * @memberof SplitBarComponent
      */
     private startPoint!: number;
 
     /**
      * @hidden @internal
      */
     public get prevButtonHidden() {
-        return this.siblings[0].hidden && !this.siblings[1].hidden;
+        return this.siblings[0].collapsed && !this.siblings[1].collapsed;
     }
 
     /**
@@ -175,9 +184,12 @@ export class IgxSplitBarComponent {
      * @hidden @internal
      */
     public get nextButtonHidden() {
-        return this.siblings[1].hidden && !this.siblings[0].hidden;
+        return this.siblings[1].collapsed && !this.siblings[0].collapsed;
     }
 
+    /**
+     * @hidden @internal
+     */
     public onDragStart(event: IDragStartEventArgs) {
         if (this.resizeDisallowed) {
             event.cancel = true;
@@ -187,6 +199,9 @@ export class IgxSplitBarComponent {
         this.moveStart.emit(this.pane);
     }
 
+    /**
+     * @hidden @internal
+     */
     public onDragMove(event: IDragMoveEventArgs) {
         const isHorizontal = this.type === SplitterType.Horizontal;
         const curr =  isHorizontal ? event.pageX : event.pageY;
@@ -200,20 +215,23 @@ export class IgxSplitBarComponent {
 
     protected get resizeDisallowed() {
         const relatedTabs = this.siblings;
-        return !!relatedTabs.find(x => x.resizable === false || x.hidden === true);
+        return !!relatedTabs.find(x => x.resizable === false || x.collapsed === true);
     }
 
+    /**
+     * @hidden @internal
+     */
     public onCollapsing(next: boolean) {
         const prevSibling = this.siblings[0];
         const nextSibling = this.siblings[1];
         let target;
         if (next) {
             // if next is clicked when prev pane is hidden, show prev pane, else hide next pane.
-            target = prevSibling.hidden ? prevSibling : nextSibling;
+            target = prevSibling.collapsed ? prevSibling : nextSibling;
         } else {
             // if prev is clicked when next pane is hidden, show next pane, else hide prev pane.
-            target = nextSibling.hidden ? nextSibling : prevSibling;
+            target = nextSibling.collapsed ? nextSibling : prevSibling;
         }
-        this.togglePane.emit(target);
+        target.toggle();
     }
 }