Bug 10165 - Vox: Master Bug
Summary: Vox: Master Bug
Status: RESOLVED CONFIGURATION CHANGED
Alias: None
Product: WebKit
Classification: Unclassified
Component: HTML Editing (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P2 Major
Assignee: Nobody
URL: http://www.vox.com/compose/
Keywords: NeedsReduction
Depends on: 12250 11826 12920 12921 12922 13116
Blocks:
  Show dependency treegraph
 
Reported: 2006-07-30 19:52 PDT by Lucas Wagner
Modified: 2021-10-20 11:32 PDT (History)
11 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Lucas Wagner 2006-07-30 19:52:52 PDT
The "compose" feature of blog site Vox doesn't work fully under WebKit/Safari; there's some level of functionality, but some features (insertion of media, strikethrough, select all, etc.) don't work as expected.

Note: since Vox is currently an invite-only service, I've created an account for anyone wishing to investigate this:

Email: heylukee@gmail.com
Password: webkit
Comment 1 Lucas Wagner 2006-07-30 20:41:36 PDT
After downloading the latest build, things are much better; resizing text seems to be the only thing not yet working.
Comment 2 Alexey Proskuryakov 2006-07-30 21:21:52 PDT
Also, text alignment doesn't seem to work.
Comment 3 Shawn Smith 2006-09-29 10:56:56 PDT
Can we get this put on the "Compatibility Hit List"?

Other issues I see:

No way to move images which are inserted using cut and paste.  Using Firefox you can drag the images within the text.
Comment 4 Shawn Smith 2006-10-14 15:24:28 PDT
Did some investigation and found these two errors in the java script console.  Probrably going to be more but this might get an investigation started??  I noted the error and the code snipit it occured in.  The <<<< desigates the line where the error occured.

ERROR #1: Null Value Error
this.savedSelection = selection.getRangeAt(0).cloneRange();

saveSelection:function(){
var selection = this.getSelection();
if(selection.createRange){
var range = selection.createRange();
if(range.parentElement){
var element = range.parentElement();
if(element && element.ownerDocument === this.document)
this.savedSelection = range.getBookmark();
}
}else if(selection.getRangeAt)
this.savedSelection = selection.getRangeAt(0).cloneRange(); <<<<
},


ERROR#2:  Value undefined (result of expression selection.removeAllRanges() is not object.
selection.removeAllRanges();

restoreSelection:function(){
this.focus();
if(!this.savedSelection)
return;
var selection = this.getSelection();
if(selection.createRange){
var range = this.document.body.createTextRange();
range.moveToBookmark(this.savedSelection);
range.select();
}else if(selection.getRangeAt){
selection.removeAllRanges(); <<<<
selection.addRange(this.savedSelection);
}
this.savedSelection = null;
this.logSelection();
},
Comment 5 Shawn Smith 2006-10-14 15:28:51 PDT
Suggest changing the name of this bug from "Compose Feature of Vox does not work" to "Compose new Blog Post on Vox is broken in Safari".

Also suggest changing the Component to HTML DOM.

Finally, suggest a higher severity given the number of VOX users who are negatively impacted who use OS X.
Comment 6 Mark Rowe (bdash) 2006-12-12 21:49:26 PST
Trying to track this as a single "This big and complex JavaScript site doesn't work" is impractical.  The issues that make it "broken" really need to be filed as separate bugs, and reduced so that they can be resolved independently.
Comment 7 Shawn Smith 2006-12-12 22:51:17 PST
Ok agreed.  I will start opening a series of individual issues and link them here.  Thanks for helping to get this some attention.
Comment 8 Mark Rowe (bdash) 2006-12-12 22:53:11 PST
It would be ideal to use this as a master bug to track the functionality of Vox.  Put the new bug numbers into the "Bug 10165 depends on:" field to track them as dependencies of this bug.
Comment 9 Shawn Smith 2006-12-13 23:45:49 PST
Bug 11826 (Lack of removeAllRanges() support in Webkit) has been opened.  It should be linked as a dependency.
Comment 10 Mark Rowe (bdash) 2006-12-13 23:55:08 PST
Shawn, you should be able to mark it as a dependency by putting the bug number in the "Bug 10165 depends on" field above the "Additional Comments" text area.
Comment 11 Justin Garcia 2007-02-27 18:15:56 PST
Re-titling as composing in Vox is no longer "broken".  Adding a subtask to cover the control selections issue.
Comment 20 Alexey Proskuryakov 2021-10-20 11:28:04 PDT
Not actionable after 15 years, closing.