UI: Add allocation directory rendering #5873
Commits on Jun 21, 2019
-
-
-
-
-
-
-
-
-
-
-
Add scenario for stable preview demonstration
This doesn’t work in every instance; sometimes when I refresh there’s a failure. I’ll have to tune the models created to eliminate these occasional problems.
Commits on Jun 24, 2019
-
-
-
-
-
Add navigation into directories
This will surely break under some circumstances; refinement to come.
-
-
-
-
-
-
-
-
-
Add assertions on complete breadcrumbs text
This isn’t truly necessary but I think it makes it easier to understand the test.
-
-
-
-
-
-
Add extensions to files in mock structure
This lets the mock endpoint be more general.
-
-
-
Commits on Jun 25, 2019
-
-
-
-
-
-
Add title for raw file modification datetime
I’d normally have assertions for this but getting it out of the hardcoded Mirage route handler is Too Much
☹️ -
This may not make sense with the file viewer… TBD
-
-
-
-
-
-
Commits on Jun 26, 2019
-
-
Update Mirage scenario to work reliably
It would periodically display an error when visiting a deep link into the allocation filesystem.
-
-
Update routing so …/fs shows root directory
This is a bit strange but I couldn’t figure out another way apart from extracting to a component, and it seemed like a shame to lose the built-in functionality of routes.
-
-
-
-
-
-
-
-
-
-
Convert task subnav partial to component
This will allow the component to encapsulate route-checking logic to show the files link as active both at root and within subdirectories.
-
Commits on Jun 27, 2019
-
Add prototype loading animation
This only applies to directory entries and not breadcrumbs, where it would either cause overlap or maybe weird jumping, but it’s more for demonstration/conversation. Thanks to @StephanWagner for the examples here: https://stephanwagner.me/only-css-loading-spinner
-
-
Add loading animation for breadcrumb links
I thought I would hate the jumping when the animation appears but it actually seems fine!
-
In the absence of a flex-grow-like quality for tables, this handles more viewport widths.
-
-
-
-
-
Add handling for fs/ API errors
The API returns 500 when a file or directory is not found, with the “no such file or directory” in the response, so this converts it to a 404 based on those qualities. TBD: is the response the same across platforms?
-
-
-
-
Refactor breadcrumbs computed property
This is still a bit more convoluted than I’d like, which is maybe the tragic outcome of using reduce… should I just make it a for loop? 😢
-
-
-
-
-
-
Remove FIXME re Mirage fs endpoint
I feel like there’s no need to add any abstraction here.
-
Change test to use hardcoded task name
This slightly improves readability, IMO.
Commits on Jun 28, 2019
-
-
Rename task property in controller
Since there’s no partial anymore, it seems more clear to not have a model property anymore when there are so many properties to work with.
-
-
-
-
-