Menu ▾ ▴

#377 Add a busy indicator to the taskbar

v1.1.x
closed
Erik Hänel
None
v1.1.1
New Functionality
2018-11-10
2018-09-13
Erik Hänel
No

Add a busy indicator to the status bar. This can be done via a simple text switch but it would be more neat, if it could be done via a wxGauge, which is green for ready and pulsing for busy.

Analysis:

The status bar fields have to be adapted, so that the first field can be used as busy indicator. If the size of the first field is defined statically, we don't need to handle the gauge in a size event handler. To separate the status bar logic from the remaining frame, we can abstrahize the status bar logic into a separate class derived from wxStatusBar.

Implementation:

  • Implementation: The new functionality was implemented as proposed by the analysis. A new class was created, which abstrahizes the status bar logics
  • Revision: [r394]
  • Implementation test: The busy indicator was invoked with every possible command combination and no deviation was detected.

Documentation:

  • ChangesLog: updated
  • Comments: New code sections were commented sufficiently.
  • Documentation articles: Not needed - internal change
  • Language files: Not needed - internal change

Tests:

This functionality was tested thoroughly in production tests. All deviations were resolved.

Related

Commit: [r394]

Discussion

  • Erik Hänel

    Erik Hänel - 2018-09-13
    • status: open --> accepted
     

  • Erik Hänel

    Erik Hänel - 2018-09-13
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -1 +1,18 @@
 Add a busy indicator to the status bar. This can be done via a simple text switch but it would be more neat, if it could be done via a `wxGauge`, which is green for  ready and pulsing for busy.
+
+###Analysis:
+(*Describe, what's the issue and which changes have to be made*)
+
+###Implementation:
+* Implementation: (*Describe, what you've changed*) 
+* Revision: [rXXX]
+* Implementation test: (*Describe the type of test, which you performed, and if it was successful*)
+
+###Documentation:
+* ChangesLog: (*Have you updated the changes log?*)
+* Comments: (*Have you written comments in the code, which describe your change?*)
+* Documentation articles: (*Have you updated the documentation articles?*)
+* Language files: (*Have you updated the language files?*)
+
+###Tests:
+(*Describe, which tests you performed and their outcome*)
    • status: accepted --> analyzing
     

  • Erik Hänel

    Erik Hänel - 2018-09-13
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -1,7 +1,7 @@
 Add a busy indicator to the status bar. This can be done via a simple text switch but it would be more neat, if it could be done via a `wxGauge`, which is green for  ready and pulsing for busy.

 ###Analysis:
-(*Describe, what's the issue and which changes have to be made*)
+The status bar fields have to be adapted, so that the first field can be used as busy indicator. If the size of the first field is defined statically, we don't need to handle the gauge in a size event handler. To separate the status bar logic from the remaining frame, we can abstrahize the status bar logic into a separate class derived from `wxStatusBar`.

 ###Implementation:

 * Implementation: (*Describe, what you've changed*)
    • status: analyzing --> implementing
     

  • Erik Hänel

    Erik Hänel - 2018-09-13
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -4,15 +4,15 @@
 The status bar fields have to be adapted, so that the first field can be used as busy indicator. If the size of the first field is defined statically, we don't need to handle the gauge in a size event handler. To separate the status bar logic from the remaining frame, we can abstrahize the status bar logic into a separate class derived from `wxStatusBar`.

 ###Implementation:
-* Implementation: (*Describe, what you've changed*) 
-* Revision: [rXXX]
-* Implementation test: (*Describe the type of test, which you performed, and if it was successful*)
+* Implementation: The new functionality was implemented as proposed by the analysis. A new class was created, which abstrahizes the status bar logics
+* Revision: [r394]
+* Implementation test: The busy indicator was invoked with every possible command combination and no deviation was detected.

 ###Documentation:
-* ChangesLog: (*Have you updated the changes log?*)
-* Comments: (*Have you written comments in the code, which describe your change?*)
-* Documentation articles: (*Have you updated the documentation articles?*)
-* Language files: (*Have you updated the language files?*)
+* ChangesLog: updated
+* Comments: New code sections were commented sufficiently.
+* Documentation articles: Not needed - internal change
+* Language files: Not needed - internal change

 ###Tests:
 (*Describe, which tests you performed and their outcome*)
    • status: implementing --> testing
     

    Related

    Commit: [r394]

  • Erik Hänel

    Erik Hänel - 2018-11-10
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -15,4 +15,4 @@

 * Language files: Not needed - internal change

 ###Tests:
-(*Describe, which tests you performed and their outcome*)
+This functionality was tested thoroughly in production tests. All deviations were resolved.
    • status: testing --> closed
     

Anonymous
Anonymous

Add attachments
Cancel





MongoDB Logo MongoDB