Closed Bug 154049 Opened 23 years ago Closed 23 years ago

URL field in event dialog

Categories

(Calendar :: General, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: chris, Assigned: chris)

Details

Attachments

(3 files)

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.
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
The backend now supports setting/getting URLs. Use event.url to do so. Code is in CVS.
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?
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.
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.
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
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.

Attachment

General

Created:
Updated:
Size: