Link to screenshots page (demo video):
Week 3: Feb 11th (Demo Day 1)
Researched Backbone models.
Completed task 3. However, this introduced a bug: On a bigger screen, the floating banner is visible and stays in a fixed position until it starts floating.
Velocity: 3
Features Added:
- I changed the banner so that it only gets added once to the diff view, instead of once per diff file.
- I redesigned the styling so that the banner properly hides when the user is at the top.
- I added a white box that contains the diff file.
- I added some divs to the diff view that will be necessary in the future.
- I changed the banner color to match the review request body. The banner is now egg white with a black border color.
- I added a dropdown button on the right side of the banner (it currently lacks functionality).
- I removed this custom made styling class and reused the classes that have already been setup for this in diffviewer.less.
- I added a new DiffFileIndexModel class, but it is currently not connected to the views. This model class is still a work in progress.
I have created a new task list to keep the project organized:
- [x] 1. Move the dropdown button to the right side of the file index view in the floating banner. (2)
- [x] 2. Add functionality to the dropdown button. When clicked, it should display the full list of diff files so the user can navigate to the file they want. (5)
- [x] 3. Add a DiffFileIndexModel that contains the state on what diff files are in the view, or if they are loading. Make both the FloatingBannerView and the DiffFileIndexView grab data from this model. This requires rearchitecting how the DiffFileIndexView works. (8)