Skip to content

Commit fab8c44

Browse files
author
Parth Shah
committed
Merge branch 'dev'
2 parents 3b7c9b2 + e3bbf7b commit fab8c44

28 files changed

+202
-72
lines changed
Lines changed: 9 additions & 0 deletions
Loading

src/components/ColorSelector/ColorSelector.jsx

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
import React, {PropTypes} from 'react'
22
import './ColorSelector.scss'
3-
import cn from 'classnames'
43
import { SketchPicker } from 'react-color'
54
import _ from 'lodash'
65
import { HOC as hoc } from 'formsy-react'
76
import {PROJECT_MAX_COLORS} from '../../config/constants'
7+
import { Icons } from 'appirio-tech-react-components'
88

99
class ColorSelector extends React.Component {
1010

@@ -40,14 +40,18 @@ class ColorSelector extends React.Component {
4040

4141
return (
4242
<div className="colorSelector">
43-
{colors.map((color) =>
43+
{value.map((color) =>
4444
<a
4545
key={color}
4646
href="javascript:"
4747
onClick={() => onColorToggle(color)}
48-
className={cn('color-card', {selected: value.indexOf(color) !== -1})}
48+
className="color-card"
4949
style={{backgroundColor: color}}
50-
/>
50+
>
51+
<span className="remove-color">
52+
<Icons.CloseIcon />
53+
</span>
54+
</a>
5155
)}
5256

5357
{colors.length < PROJECT_MAX_COLORS &&

src/components/ColorSelector/ColorSelector.scss

Lines changed: 15 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,21 @@
99
border: 1px solid $tc-gray-60;
1010
border-radius: 4px;
1111
margin: 0 15px;
12-
&.selected {
13-
border: 4px solid $tc-gray-60;
12+
position: relative;
13+
.remove-color {
14+
background-color: $tc-gray-60;
15+
opacity: 0.7;
16+
display: none;
17+
position: absolute;
18+
width: 39px;
19+
height: 38px;
20+
}
21+
&:hover {
22+
.remove-color {
23+
display: flex;
24+
justify-content: center;
25+
align-items: center;
26+
}
1427
}
1528
}
1629
.color-card-add{

src/components/LinksMenu/DeleteLinkModal.jsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ import React, {PropTypes} from 'react'
22

33
const DeleteLinkModal = ({ onCancel, onConfirm}) => {
44
return (
5-
<div className="modal">
5+
<div className="modal delete-link-modal">
66
<div className="modal-title danger">
77
You're about to delete a link
88
</div>

src/components/LinksMenu/LinksMenu.jsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,12 @@ import uncontrollable from 'uncontrollable'
77
import cn from 'classnames'
88

99
const LinksMenu = ({ links, limit, canDelete, isAddingNewLink, onAddingNewLink, onAddNewLink, onChangeLimit, linkToDelete, onDeleteIntent, onDelete }) => (
10-
<Panel className={cn({'modal-active': isAddingNewLink})}>
10+
<Panel className={cn({'modal-active': (isAddingNewLink || linkToDelete >= 0) })}>
1111
{!isAddingNewLink && <Panel.AddBtn onClick={() => onAddingNewLink(true)}>Create New Link</Panel.AddBtn>}
1212
{!isAddingNewLink && <Panel.Title>
1313
Links ({links.length})
1414
</Panel.Title>}
15+
{ (isAddingNewLink || linkToDelete >= 0) && <div className="modal-overlay" />}
1516
{isAddingNewLink &&
1617
<AddLink
1718
onAdd={(link) => {

src/components/LinksMenu/LinksMenu.scss

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,4 +33,8 @@
3333
line-height: $base-unit*6;
3434
padding-left: $base-unit*4;
3535
}
36+
37+
.delete-link-modal {
38+
margin-left: -4*$base-unit;
39+
}
3640
}

src/components/LoadingIndicator/LoadingIndicator.jsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import React from 'react'
33
require('./LoadingIndicator.scss')
44

55
const LoadingIndicator = () => {
6-
const backgroundImageUrl = { backgroundImage: `url(${require('./ripple.svg')})` }
6+
const backgroundImageUrl = { backgroundImage: `url(${require('./loader.gif')})` }
77

88
return (
99
<div
Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
.loading-indicator {
2-
height: 100px;
3-
width: 100px;
4-
margin: 150px auto;
5-
background-repeat: no-repeat;
6-
background-position: center;
2+
height: 80%;
3+
width: 100%;
4+
background: 50% 50% no-repeat;
5+
background-size: 70px 70px;
76
}
106 KB
Loading

src/components/LoadingIndicator/ripple.svg

Lines changed: 0 additions & 1 deletion
This file was deleted.

0 commit comments

Comments
 (0)