[WIP] Add Show/Hide .min button for minified source files

Review Request #7700 — Created Oct. 15, 2015 and discarded — Latest diff uploaded

eklau
Review Board
master
reviewboard

In order to determine if the file is a minified file, I have to change the file in the database so that the will end with .min.css, .min.js, or .min.less.

First, I went to localhost:8080. Then I went to the top right bar, under my username I clicked into admin. Then, I went to database near the top left navigation bar, under DIFFVIEWER, I clicked into file diff and changed various file names to file.min.js and file.min.css and file.min.less on both source and destination file to test.

Now there are some files with keywords ending with ".min.", I can test to modify files with those substring.
-10/19

I am trying to figure out how the diffReviewableView.js is connected with diff_file_fragment.html to link the onclick events with the button.

I also need to know what class the contents are being wrapped around so that I can show/hide those contents when I click the button.
-10/24

The new Show/Hide .min button can now show and hide the diff-box. The location of the button is now updated. I added the button to where the other expand/collapse buttons were for managing data within the diff-box. I still need to only show the new button for .min.css, .min.js, and .min.less files.
-10/31

I have to add a function in the python code so that it will recognize that the file name is a minified file. I'm still working on understanding the Python source code for whitespace so that I can have a similar message for the minified file. I need to hide the contents of the minified files by default, and have a link that when pressed, will show those contents.
-11/6

The diff review page now shows the message to detect that the file is a minified file. I still have to change the css for the message (currently uses whitespace class). I also need to make hide the contents of the minified file by default and create a link that will show the contents of the file when pressed.
-11/14

Test string inputs to see where they are being displayed
-10/19

Test button to see where they are displayed. Test button functionality with alert()
Test button functionality with diffReviewable.js
-10/24

Add button location with view_diff.html
Test button functionality with diffViewerPageView.js
Remove diff-highlight when .min file is hidden
Change font-weight of Show/Hide .min button to normal weight
-10/31

Working on understanding python source code for whitespace, no code updated yet
-11/6

Add new member of the dictionary in diff_utils.py
Add .min. file message in diff_file_fragment.html
Test min-link class in diffReviewable.js
Remove old button and it's functions
-11/14


    Loading...