Gui progress » History » Version 37
Adrian Georgescu, 07/19/2009 12:10 PM
1 | 1 | Adrian Georgescu | [[TOC(gui_progress, depth=3)]] |
---|---|---|---|
2 | |||
3 | = Critical issues = |
||
4 | |||
5 | 5 | Adrian Georgescu | Must be solved first. |
6 | 1 | Adrian Georgescu | |
7 | 35 | Adrian Georgescu | 1. Sessions remain indefinitely in the drawer |
8 | 37 | Adrian Georgescu | 1. Closed session keeps updating the session counter |
9 | 1 | Adrian Georgescu | 1. Voice calls do not update the status of the drawer |
10 | 37 | Adrian Georgescu | 1. Incoming sessions do no cancel and session fails when attempting to reject/accept later |
11 | 35 | Adrian Georgescu | |
12 | {{{ |
||
13 | 2009-07-19 12:07:27.968 Blink[43601:10b] File "/Users/adigeo/work/blink/build/Debug/Blink.app/Contents/Resources/ContactWindowController.py", line 462, in sip_session_incoming |
||
14 | 2009-07-19 12:07:27.969 Blink[43601:10b] session.reject(resp_code) |
||
15 | 2009-07-19 12:07:27.969 Blink[43601:10b] File "<string>", line 1, in reject |
||
16 | 2009-07-19 12:07:27.969 Blink[43601:10b] File "/Library/Python/2.5/site-packages/sipsimple/session.py", line 46, in wrapper |
||
17 | 2009-07-19 12:07:27.971 Blink[43601:10b] raise RuntimeError('cannot call %s in %s state' % (func.__name__, obj.state)) |
||
18 | 2009-07-19 12:07:27.971 Blink[43601:10b] RuntimeError: cannot call reject in terminated state |
||
19 | }}} |
||
20 | |||
21 | 1. Incoming sessions to not display audio session information |
||
22 | 31 | Adrian Georgescu | 1. In chat window, after entering a message the scrollbar '''moves up by itself''' |
23 | 1. When text is pasted, as opposed to typed on the keyboard, the '''pasted content''' is sent but not displayed |
||
24 | 11 | Adrian Georgescu | 1. Display the '''negotiated codec''' next to audio and use colors from email snapshot |
25 | 36 | Adrian Georgescu | 1. Align '''session duration''' to the left |
26 | 29 | Adrian Georgescu | 1. When enable a new account, the main interface displays '''no account selected''' anymore |
27 | 1. On startup, '''select the default account''' in the account lists |
||
28 | 1. When select a different account, mark it as '''default''' |
||
29 | 11 | Adrian Georgescu | 1. '''Expand the session frame horizontally''' to fill on the drawer to its full width and draw just one line between two voice calls |
30 | 4 | Adrian Georgescu | 1. '''Stun server''' addresses cannot be set, they seem to inherit the results found in DNS |
31 | 1 | Adrian Georgescu | 1. '''general.rtp.local_ip''': Can't set option local_ip illegal local IP address value: auto |
32 | 1. '''general.rtp.port_range''': (AttributeError: 'PortRangeOption' object has no attribute 'save') |
||
33 | 1. '''general.sip.transports''' cannot be saved |
||
34 | 10 | Adrian Georgescu | 1. '''general.audio.codec_list''' cannot be saved and displays duplicate codecs |
35 | 18 | Adrian Georgescu | 1. '''account.audio.codec_list''' cannot be saved and displays duplicate codecs |
36 | 23 | Adrian Georgescu | 1. '''account.msrp.relay''' cannot be set: global name 'MSRPRelayAddres' is not defined |
37 | 25 | Adrian Georgescu | 1. When '''disable account''' in preferences, it does not get removed from the main interface account lists |
38 | 30 | Adrian Georgescu | 1. Display '''active account in bold text''' |
39 | 23 | Adrian Georgescu | |
40 | 32 | Adrian Georgescu | = Sessions = |
41 | 30 | Adrian Georgescu | |
42 | 32 | Adrian Georgescu | * Center the '''drawer title''' and make it look like a title bar |
43 | * Show the '''drawer maximized''' horizontally |
||
44 | * Center align '''Hangup All and Conference buttons''' |
||
45 | * Only one audio session can be '''active at a time''', when answering a new audio session, all other existing audio sessions must be put on hold |
||
46 | 33 | Adrian Georgescu | * When the active session is selected with the mouse, what is typed must be transmitted as DTMF tones |
47 | 32 | Adrian Georgescu | * When an incoming request has '''chat only''' and the from header is in the contact list, automatically accept the session |
48 | * Highlight the active session with a '''bold''' border |
||
49 | * When a session has ended, display it for '''5 more seconds''' then hide it and shuffle the deck |
||
50 | * If all sessions have ended, the '''drawer must be closed''' |
||
51 | * Add an item to the '''Session menu''' to show the drawer, if it was closed |
||
52 | |||
53 | 1 | Adrian Georgescu | = Chat window = |
54 | |||
55 | * Change title of the chat window to '''remote@domain''' |
||
56 | * '''Hide the empty scroll bar''' of the text input box |
||
57 | * Display a status bar on the bottom of the window and '''show the connection information''' in it instead of the main window |
||
58 | * Display the Chat recipient in a '''tab at the bottom of the window''' |
||
59 | * Play an audible message when a message is received, based on '''silent''' and '''general.message_received_sound''' settings |
||
60 | * Play an audible message when a message is sent, based on silent and '''general.message_sent_sound''' settings |
||
61 | 4 | Adrian Georgescu | * '''Throttle playback''' of audible notifications to maximum one every 3 seconds |
62 | 1 | Adrian Georgescu | * Append chat messages to '''chat.history_directory/sip_account/YYYYMMDD-recipient.txt''' |
63 | * For each message that appears in the history window: |
||
64 | * First display a rounded thin rectangle with the '''recipient''' on the left and '''full date''' on the right side of the text in history with either light blue or grey background depending on which party is typing it , display the full FROM headers |
||
65 | * Then display the content of the message |
||
66 | * When dragging a recipient tab outside of the window '''spawn a new chat window''' and move the session into it |
||
67 | 2 | Adrian Georgescu | * Display the number of '''un-read messages''' in non-visible tabs in a red circle over the tab name |
68 | 1 | Adrian Georgescu | |
69 | 6 | Adrian Georgescu | = Contacts = |
70 | 21 | Adrian Georgescu | |
71 | 22 | Adrian Georgescu | * '''Hide display name''', show only the SIP address in the account selection |
72 | 21 | Adrian Georgescu | * Remove the blue '''is-selected''' rectangle that appears around the active account |
73 | * '''Replace''' 3333 with 3333@sip2sip.info and 4444 with 4444@sip2sip.info |
||
74 | 1 | Adrian Georgescu | * When searching contacts, do not display '''Add button''' if the contact already exists |
75 | 22 | Adrian Georgescu | * Show the '''Display name''' if set for the Contact, otherwise the SIP address |
76 | 19 | Adrian Georgescu | * Add new '''contact window fields''': SIP address, Display name and Group |
77 | * '''Delete the contact''' when pressing delete, show a confirm window |
||
78 | * When '''searching a contact''' and starting a session to it: (NameError: global name 'row' is not defined) |
||
79 | |||
80 | = Preferences = |
||
81 | 24 | Adrian Georgescu | |
82 | 1 | Adrian Georgescu | * Properly '''align''' horizontally the Advanced settings |
83 | 30 | Adrian Georgescu | * In advanced sections, display '''Enabled:''' on the left side of the checkbox instead of the right |
84 | * Delete the '''little title bar on top of the account list''' |
||
85 | * '''Delete account Enabled checkbox''', leave only the checkbox in the account list |
||
86 | 1 | Adrian Georgescu | * Hide the vertical and horizontal '''scroll bars''' when no scrollbar is needed |
87 | * '''De-register''' the account and remove it from the account list in main interface when disable the account |
||
88 | * When '''enable an account''', select it in the main interface account list |
||
89 | * Add a '''Reset option''' to reset advanced settings to default values |
||
90 | 30 | Adrian Georgescu | * Display the '''bonjour account''' always at the end of the list |
91 | * Cannot '''delete Bonjour''' account: (TypeError: cannot delete BonjourAccount instance with default id). Fo not allow the user to attempt to delete it. Bonjour can only be enabled/disabled but not deleted. |
||
92 | 1 | Adrian Georgescu | * For '''account.ringtone''' advanced setting, the default setting must be inherited from '''general.ringtone.inbound''' |
93 | * '''general.message_received_sound''' does not fit in window |
||
94 | * Allow ordering of accounts in the account list |
||
95 | |||
96 | Hide the following settings, they should not be editable from GUI: |
||
97 | |||
98 | * global.chat.accept_types |
||
99 | * global.chat.accept_wrapped_types |
||
100 | * global.sip.ignore_missing_ack |
||
101 | * account.presence.publish_interval |
||
102 | 19 | Adrian Georgescu | * account.presence.subscribe_interval |
103 | * account.presence.subscribe_rls_services |
||
104 | * account.presence.subscribe_xcap_diff |
||
105 | |||
106 | Display SIP registration state in '''account.advanced.registration''' section. |
||
107 | 29 | Adrian Georgescu | |
108 | = Engine = |
||
109 | |||
110 | * If '''engine is dead''', the message '''no sip account is active''' is displayed but is unclear what the real reason is. An engine stop must be logged to general debug window |
||
111 | * On startup, check if the audio device from the preferences '''is available''' in the detected devices list. If not, set the device to the system default device and display a warning to the user to set the devices in the preferences |
||
112 | |||
113 | * If the engine has stopped (because of some crash) the preferences do not show anymore: |
||
114 | |||
115 | {{{ |
||
116 | Blink[26992:10b] <type 'exceptions.AttributeError'>: 'Engine' object has no attribute 'output_devices' |
||
117 | }}} |
||
118 | |||
119 | By setting the wrong TLS certificate files, the middleware does not start anymore: |
||
120 | |||
121 | {{{ |
||
122 | [Session started at 2009-07-19 10:37:39 +0200.] |
||
123 | Traceback (most recent call last): |
||
124 | File "/Users/adigeo/work/blink/build/Debug/Blink.app/Contents/Resources/BlinkAppDelegate.py", line 40, |
||
125 | in applicationDidFinishLaunching_ |
||
126 | self.backend.init(options) |
||
127 | File "/Users/adigeo/work/blink/build/Debug/Blink.app/Contents/Resources/SIPManager.py", line 108, in init |
||
128 | self._app.start(config_backend=config_be) |
||
129 | File "/Library/Python/2.5/site-packages/sipsimple/api.py", line 85, in start |
||
130 | trace_sip=settings.logging.trace_sip, |
||
131 | File "/Library/Python/2.5/site-packages/sipsimple/engine.py", line 75, in start |
||
132 | self._ua = PJSIPUA(self._handle_event, **init_options) |
||
133 | File "core.ua.pxi", line 66, in sipsimple.core.PJSIPUA.__init__ (sipsimple/core.c:23189) |
||
134 | File "core.lib.pxi", line 106, in sipsimple.core.PJSIPEndpoint.__cinit__ (sipsimple/core.c:6231) |
||
135 | File "core.lib.pxi", line 167, in sipsimple.core.PJSIPEndpoint._start_tls_transport (sipsimple/core.c:6884) |
||
136 | PJSIPError: Could not create TLS transport: Error loading/verifying SSL CA list file (PJSIP_TLS_ECACERT) |
||
137 | None |
||
138 | }}} |
||
139 | 1 | Adrian Georgescu | |
140 | 34 | Adrian Georgescu | |
141 | 1 | Adrian Georgescu | = Presence bar = |
142 | |||
143 | Make it as high as the contacts (two rows high). |
||
144 | |||
145 | 2 | Adrian Georgescu | * Show '''own photo''' to the left |
146 | * Show '''Display name''' next to the photo on top row |
||
147 | 1 | Adrian Georgescu | * Show '''presence activity''' next to name on top row |
148 | * Show '''activity note''' on second row |
||
149 | 7 | Adrian Georgescu | |
150 | = Debug window = |
||
151 | |||
152 | * Display all traces in a '''debug window''' with tabs for each debug type |
||
153 | * Add a '''General messages''' window to the Debug menu, first item |
||
154 | 1 | Adrian Georgescu | |
155 | = GUI settings = |
||
156 | |||
157 | Remember the following settings between restarts: |
||
158 | |||
159 | 28 | Adrian Georgescu | * '''Main window''' size and screen position (done) |
160 | 2 | Adrian Georgescu | * '''Debug window''' and the selected tabe |
161 | 1 | Adrian Georgescu | * Status of '''contact groups''' - colapsed/expanded and their order |
162 | 17 | Adrian Georgescu | * SIP '''accounts order''' list |
163 | 14 | Adrian Georgescu | |
164 | = Middleware changes = |
||
165 | |||
166 | 34 | Adrian Georgescu | * Check the validity of the TLS certificate before starting the engine |
167 | 14 | Adrian Georgescu | * Move '''account.outbound_proxy''' to '''account.sip.outbound_proxy''' |
168 | * It seems that the stun servers are learned from the DNS and cannot be overwritten in settings, they show up automatically |
||
169 | * if I set the relay to abcd.com you must use the default standard port of 2855, now is set to zero: relay = abcd.com:0;transport=tls |
||
170 | * Append dummy to the lists of available audio devices |
||
171 | * We need a solution to accept a session without a 180- ringing. For instance we can have automatic answer based on various preferences. |
||
172 | * Move '''general.audio.codec_list''' to '''general.rtp.audio_codecs''' |
||
173 | * Move '''account.audio.codec_list''' to '''account.rtp.audio_codecs''' |
||
174 | * Move '''account.audio.srtp_encryption''' to '''account.rtp.srtp_encryption''' |
||
175 | * Delete '''account.audio''' section |