Closed
Bug 154049
Opened 23 years ago
Closed 23 years ago
URL field in event dialog
Categories
(Calendar :: General, enhancement)
Calendar
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: chris, Assigned: chris)
Details
Attachments
(3 files)
1.58 KB,
patch
|
Details | Diff | Splinter Review | |
747 bytes,
patch
|
Details | Diff | Splinter Review | |
1.46 KB,
patch
|
Details | Diff | Splinter Review |
Since events can have a URI attached to them, we should have the ability to set,
modify, and view said URIs. I don't know if we have support for this in the
backend, if we do please feel free to change this to a front-end bug.
Assignee | ||
Comment 1•23 years ago
|
||
Assignee | ||
Comment 2•23 years ago
|
||
Assignee | ||
Comment 3•23 years ago
|
||
I still need to write up some JavaScript code to pop up a new Navigator window
when the "Visit URL" button is clicked. Also, code is still needed on the
backend so that we can get/set the URI field for events and todos.
my first patch there will be put to pasture as soon as I have the JS code for
opening a window, since I neglected to add a handler for the button.
Status: NEW → ASSIGNED
Comment 4•23 years ago
|
||
The backend now supports setting/getting URLs. Use event.url to do so. Code is
in CVS.
Assignee | ||
Comment 5•23 years ago
|
||
With the changes in this patch, when I try to set url, I get this in the JS
Console:
Error: uncaught exception: [Exception... "Cannot modify properties of a
WrappedNative" nsresult: "0x80570034 (NS_ERROR_XPC_CANT_MODIFY_PROP_ON_WN)"
location: "JS frame :: chrome://calendar/content/calendarEventDialog.js ::
onOKCommand :: line 277" data: no]
Is this a fault with my patch, or do we have a backend problem?
Comment 6•23 years ago
|
||
You need to rebuild your libxpical stuff in order to use Mostafa's changes.
The patch looks good to me, just add in some code to open a browser when you
click the button and you're done.
Assignee | ||
Comment 7•23 years ago
|
||
I can't get as far as rebuilding libxpical, since libical itself still hates me
and my computer. But hey, if that patch is decent, then all that needs to be
added is control for the "Visit URL" button.
Comment 8•23 years ago
|
||
I've finished this up and will put it into CVS later.
Resolving it as fixed.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Comment 9•19 years ago
|
||
The bugspam monkeys have been set free and are feeding on Calendar :: General. Be afraid for your sanity!
QA Contact: colint → general
You need to log in
before you can comment on or make changes to this bug.
Description
•