Difference between revisions of "User:RomitKumar/GSoC19/logs"

(August 19)
(Week 13 (August 19 - August 25))
Line 197: Line 197:
=== August 19 ===
=== August 19 ===
Completed the code for hyperlink feature. Made pr [https://github.com/openscad/openscad/pull/3045 #3045]. Now the part left is regarding correct implementation of character location of use'd files. This feature also needs to be extended to include'd files.
Completed the code for hyperlink feature. Made pr [https://github.com/openscad/openscad/pull/3045 #3045]. Now the part left is regarding correct implementation of character location of use'd files. This feature also needs to be extended to include'd files.
=== August 20 ===
Discussed with mentor about possible solution for the erroneous behavior of Parameter Widget.

Latest revision as of 19:45, 24 August 2019


Community Bonding Period (May 6 - May 27)[edit]

May 14[edit]

Discussed about sharing of mock-up code and prototypes with mentor

May 15[edit]

Developed a prototype for gui implementation of multi-tab feature. Prototype1 Code

May 16[edit]

Implementing prototype in openscad codebase

May 17[edit]

Implemented the prototype in openscad codebase. Made PR #2949. This pr will act as a sandbox for prototyping and sharing of coding with mentor. Discussion about integrating pre-available editing features in all tabs

May 18[edit]

Implemented the comment feature using signal dispatcher as suggested by mentor. Made the pr.

May 19[edit]

Studying the openscad codebase to look for implementation of saving feature for each tab.

May 20[edit]

Implemented the feature of saving of tab contents in prototype code. Main Window name updates on switching tab.

May 21[edit]

Discussed about correctness of signal dispatcher.

May 22[edit]

Implemented signal dispatcher in a separate class. Made the pr. Discussed about implementation details of multi-tab feature. Decided to implement the feature in a separate TabManager class.

May 23[edit]

Studying the openscad codebase for feasibility of implementation of TabManager class.

May 24[edit]

Studying openscad codebase.

May 25[edit]

Made a prototype project with similar gui features as multi-tab. Implemented those features using with a separate TabManager class as suggested by mentors. Prototype2 Code

May 26[edit]

Implement basic gui(opening and closing of tabs) of multi-tab in openscad code. The feature in implemented in a separate TabManager class. Made the pr #2955

Coding Period (May 27 - Aug 19)[edit]

Week 1 (May 27 - June 2)[edit]

May 27[edit]

Discussed about possibility and various method of integration of Simple and QScintilla Editor with multi-tab editor.

May 28[edit]

Implemented the code for integration of pre-available editor features with multi tab. There are some issues that need to be handled like checking the undo state, highlighting of error in multiple files.

May 29[edit]

Removed support for legacy editor and corrected the keyboard mapping for zoom in/out. Pushed commits cfa32da and 1ad0f53 to pr #2955.

May 30[edit]

Corrected the updation of undoState and windowModified. Now these states behave correctly for multi-tab environment. Realised that stl exports and enable status of parameterWidget are not behaving correctly.

May 31[edit]

Corrected the behaviour of stl exports and enable status of parameterWidget. Running some basic tests to check the correctness of work done till now.

June 1[edit]

Discussed with mentor about behaviour of Customizer widget and animation on changing tabs. Implemented those behaviour in codebase of openscad. Pushed the commits to pr #2955

June 2[edit]

Run basic tests on gui implemented so far. Discovered that Find, Find and Replace are not behaving correctly for multi-tab.

Week 2 (June 3 - June 9)[edit]

June 3[edit]

Correcting the behaviour of Find, Find and Replace for multi-tab

June 4[edit]

Implemented the correct behaviour of Find, Find and Replace for multi-tab. Pushed commit 23129ba to pr #2955. Found a bug related to previous implementation of Find. Raised issue #2962. Discussed about implementing save feature for multi-tab and naming of new tabs and tabs that have same file names.

June 5[edit]

Studying the openscad codebase for implementation of save features in multi-tab.

June 6[edit]

Working on implementing the open feature for multi-tab.

June 7[edit]

Removed the option for single file QScintilla Editor. Instead now multi-tab editor will not show tabs in case of single tab, effectively giving the feeling of single file QScintilla Editor. Implemented this feature in multi-tab editor. Pushed commit d594980 to pr #2955

June 8[edit]

Continuing the work to implement open feature in multi-tab.

June 9[edit]

Completed the work of implementing Open(File menu option, Recent Files, Examples), Save, Close feature for multi-tab. Pushed the corresponding commits to pr #2955.

Week 3 (June 10 - June 16)[edit]

June 10[edit]

Started testing of code submitted.

June 11[edit]

Changed the behaviour of Open and Open Tab as suggested by mentors. Made commits 6a0a1d1 and e3ff4d9 in pr #2955.

June 12[edit]

Testing and cleaning of code.

June 13[edit]

Done cleanup of code. This includes renaming variable/function, removing redundant function, removing mdi checkbox from Preferences/Advanced, correcting implementation of reload, prevention of double emission of currentChanged signal for first tab. Pushed commits 0e92308 and cdaee5e in pr #2955.

June 14[edit]

Implemented the Save All feature. Pushed commit b0c95e8 in pr #2955. Corrected the enabling of Parameter Widget on file load and opening of Recent Files and examples in empty tab. Pushed commits 35e3fe0 and acc3123 corresponding to the changes. Discussed with mentor about the idea of Tab Header spanning the whole window.

June 15[edit]

Working on Tab Header spanning whole window length.

June 16[edit]

Implemented the feature of Tab Header spanning whole window length. Pushed commits 087359f and c59df44 in pr #2955.

Week 4 (June 17 - June 23)[edit]

June 17[edit]

Shifted Tab Header implementation from dock widget to toolbar. This toolbar clings to editor dock widget when it is undocked. Pushed commits fd93e57, a50fd61 and 689292c in pr #2955.

June 18[edit]

Testing multi tab implementation

June 19[edit]

Continuation of testing of multi-tab implementation. Looking and exploring various methods for auto completion feature.

June 20[edit]

Posted in mailing group inviting users for testing of multi-tab feature. Going through the codebase at pr #2889 to understand details of auto completion feature.

June 21[edit]

Updated implementation of tab tool bar such that it is hidden when a single tab is present. Pushed commit dff9b9b in pr #2955.

June 22[edit]

Looking at web resources available to understand auto-completion in qscintilla.

June 23[edit]

Made a new pr #2975 implementing auto-completion of openscad keywords. The commit is bdc8e10. Also pushed commit 7cda02a to enable qscintilla in travis check. Discussed with mentor about various options of providing keyword list for auto-completion.

Week 5 (June 24 - June 30)[edit]

June 24[edit]

Implemented ctrl+insert shortcut to display template list. Pushed commit 10bfc3f in pr #2975

June 25[edit]

Checking the plausibility of making keyword list for auto-completion at Builtin class.

June 26[edit]

Discussed with mentor about implementation details for auto-completion keyword list in Builtin class.

June 27[edit]

Pushed commit 77dfcc5 in pr #2975 implementing registering of keywords in Builtin class.

June 28[edit]

Pushed commit dc1146b in pr #2975 replacing qt structures with std structures in core classes.

June 29[edit]

Pushed commit a3e8950 and d1dc26f in pr #2975 adding other keywords in keyword list. Discussed with mentor regarding html formatting of calltips.

June 30[edit]

Working on html formatting of calltips.

Week 6 (July 1 - July 7)[edit]

July 1[edit]

Made a separate Keyword class for storing information about keywords. Pushed commit 1eefb4a in pr #2975. Checked about possibilty of html formatted string for calltips. It seems that QScintilla does not support formatting of strings in html form.

July 2[edit]

Pushed commit d9246dd in pr #2975 modifying the structure of Keyword class. Discussed with mentor about final structure for implementation of keyword auto completion.

July 3[edit]

Working on implementation of mentor suggested approach.

July 4[edit]

Applying the approach for primitives keywords.

July 5[edit]

Completing the primitive keyword autocompletion.

July 6[edit]

Pushed commits 6836128 and 0720ea1 in pr #2975 implementing auto completion for all keywords.

July 7[edit]

Pushed commit 65676bf in pr #2975 adding remaining non-registered keywords in auto-complete list.

Week 7 (July 8 - July 14)[edit]

July 13[edit]

Pushed commit 12c605f in pr #2955 correcting implementation of opening of files in empty editor tab. Also pushed commit 30085ff so that files with & are shown correctly.

July 14[edit]

Pushed commit a99d689 in pr #2955 resolving merge conflict caused due to merging of pr #2989 in master branch.

Week 8 (July 15 - July 21)[edit]

July 19[edit]

Discussed with mentor about overall feedback of work done till now. Get to know about feedback message on multi-tab regarding multi-tab.

July 20[edit]

Pushed commit 87a58ca in pr #2975 implementing templates being read from json file. Discussed about possible improvements in template feature and calltip format. Pushed commit 606f743 in pr #2955 initializing tabCount variable.

July 21[edit]

Working on improvements of template feature discussed.

Week 9 (July 22 - July 28)[edit]

July 22[edit]

Discussed with mentor about possible placeholder of cursor in templates

July 23[edit]

Pushed commit a378b23 and 92c8b2e in pr #2975 implementing improvements in template feature as discussed with mentor.

July 24[edit]

Working on deciding a calltip format for auto-completion feature

July 25[edit]

Pushed commit b280d9e in pr #2975 updating the calltip format for some keywords.

July 26[edit]

Pushed commit d6a2eec in pr #2975 adding settings in Preferences to enable/disable autocomplete and character threshold.

July 27[edit]

Pushed commit 92eaaae improving handling of file drops, commit a3fe6d2 enabling opening of multiple file from open dialog box and commit 94eac67 preventing opening of same file in different tab in pr #2955

July 28[edit]

Going through the code to analyze the possibility of extending error handling and allowing switch to tab / opening tab with error location

Week 10 (July 29 - August 4)[edit]

July 29[edit]

Continuing reading code base for extending error handling. Discussed with mentor about calltip format

July 30[edit]

Pushed commit 464ac3c in pr #2975 improving calltip format

July 31[edit]

Made pr #3022 adding never option in reload confirmation dialog box

August 1[edit]

Discussed with mentor about hotspot feature of use'd/include'd files

August 2[edit]

Going through the QScintilla documentation, trying to find a way to implement hotspot feature

August 3[edit]

Still going through the documentation. Realized that this task can be achieved by indicator. After discussion, decided to stick to hotspot

August 4[edit]

Found a bug in reading of Preferences settings. Made pr #3028 removing this bug. Discussed with mentor regarding hotspot feature, and decided to look into parser for it. A bug was reported in pre-filled file name in export dialog box.

Week 11 (August 5 - August 11)[edit]

August 5[edit]

Made pr #3029 correcting file name shown in export dialog box.

August 6[edit]

Studying the codebase and looking for possible approach to implement hyperlink.

August 7[edit]

Going through the QScintilla documentation looking for possible clues.

Week 12 (August 12 - August 18)[edit]

August 12[edit]

Discussed with mentor about a possible idea to implement hyperlinks. Planning to pass on line and column number information of use's file name in FileModule::registerUse.

August 13[edit]

Going through lexer.l and parser.y files, trying to understand their working and finding a way to pass line and column number data of use'd file.

August 14[edit]

Found LOC macro in parser.y. It appeared to be promising. However, it is giving strange results. Besides the location information changes on different preview even though there is no change in file. The code showing this behaviour is here. Checking if I have done something wrong.

August 15[edit]

Discussed with mentor and decided to skip the location information part of use'd file, assuming it provides correct data, and implement the remaining hyperlink part.

August 16[edit]

Reading about the implementation of hotspot to achieve hyperlink. It seems that the ScadLexer should be a sub-class of QsciLexerCustom, if we want hotspot links, however that is not the case. Decided to drop hotspot for hyperlink.

August 17[edit]

Reading about Indicators to implement hyperlink.

August 18[edit]

Working on code to implement hyperlink feature via Indicators.

Week 13 (August 19 - August 25)[edit]

August 19[edit]

Completed the code for hyperlink feature. Made pr #3045. Now the part left is regarding correct implementation of character location of use'd files. This feature also needs to be extended to include'd files.

August 20[edit]

Discussed with mentor about possible solution for the erroneous behavior of Parameter Widget.