Skip to content
This repository has been archived by the owner on Feb 26, 2023. It is now read-only.

[WIP] Refactor #153

Merged
merged 45 commits into from
Dec 9, 2016
Merged

[WIP] Refactor #153

merged 45 commits into from
Dec 9, 2016

Conversation

jippi
Copy link
Owner

@jippi jippi commented Dec 2, 2016 •
edited
Loading

  • move files around and rename some to be more consistent
  • update dependencies
  • add pending status allocation filter
  • use 2 space indent for js and jsx alike (most consistent)
  • remove helpers not really being shared anyway
  • add react optimize preset (nice perf bump)
  • don't force width of TASK GROUP in allocation lists

This PR can be tested as iverberk/nomad-ui:pr-153

jippi added 25 commits December 2, 2016 14:38
@jippi
move files around and rename some to be more consistent
a92acd6
@jippi
fixes #151
1098cce
@jippi
add editorconfig and fix forced indentation
eef370a
@jippi
auto-fix eslint errors
1103249
@jippi
update editor config
0b865c6
@jippi
be more strict in NomadLink on errors
3122ac3
@jippi
move each allocation row into its own component
5e22f60
@jippi
inline rendering of AllocationListRow
ebf869e
@jippi
add react-optimize as a preset
bf175c6
@jippi
don't short job id
4c2db30
@jippi
dont generate uuid for each time tooltip, but rely on the parent to p…
bdf8047 
…rovide unique identifier
@jippi
more fixes to rendering perf
e00a496
@jippi
significantly improve allocation list rendering performance
56c9c41
@jippi
cleanup perf
6ff18b0
@jippi
build and publish PR
30fd1e7
@jippi
remove existing css and further refactor the ui
680ae91
@jippi
further refactor the allocation list
44ee78e
@jippi
fix lints
e9533cf
@jippi
more UI tweaks
381aa0f
@jippi
move to yarn and cleanup js lints
ca18e13
@jippi
no more NomadLink
d08d447
@jippi
re-style jobs list
f450f73
@jippi
initial style of Job Info page
809c401
@jippi
more styling of job info page
8ef7547
@jippi
make jslint pass
9d966ba
@jippi jippi mentioned this pull request Dec 7, 2016
Closed
@jippi jippi mentioned this pull request Dec 7, 2016
Closed
@jippi jippi merged commit 1fa30fe into master Dec 9, 2016
@jippi jippi deleted the mega-refactor branch December 9, 2016 15:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jippi