Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Plain text paste new #439

Merged
merged 11 commits into from Jul 25, 2011

Conversation

Projects
None yet
2 participants
Owner

pjrobertson commented Jul 22, 2011

This is an attempt to create a tidied up #419

Hopefully this one will be easier to merge, but that is not definite...

Thanks to @skurfer for suggesting Gitbox, it gave me the idea to git cherry-pick the commits I wanted.
This should be easier to look over - which I think Rob's already done.
If it's all good, this can be merged, #419 can be closed and #430 can be considered separately to this one.

Owner

pjrobertson commented Jul 22, 2011

There is a merge conflict because of what I said in #419:

It is because, with @skurfer's htmlParsing branch merged, git thinks the lines in the xcodeproj for adding BeautifulSoup.py and Clipboard.icns are conflicting. Silly git.

Owner

skurfer commented Jul 23, 2011

There are two conflicts, but it’s easy enough to deal with. I’ll check this out and make sure it’s still good. I say close #419.

Owner

pjrobertson commented Jul 23, 2011

Merge master now eh? ;-)

Cheers

Owner

skurfer commented Jul 24, 2011

Sorry. I will look at it as soon as I can. Should have mentioned it before, but my wife is out of town until Wednesday, so it’s me and two little ones. Imagine my free time. Then add to that the Lion upgrade which requires and Xcode upgrade (which may or may not compile QS with the release config, etc.)

Short version: maybe tomorrow. :)

Owner

pjrobertson commented Jul 24, 2011

No worries, it doesn't look like we're going to get anything out this
weekend anyway :(

Good luck getting some free time! ;-)

On 24 July 2011 04:59, skurfer <
reply@reply.github.com>wrote:

Sorry. I will look at it as soon as I can. Should have mentioned it before,
but my wife is out of town until Wednesday, so its me and two little ones.
Imagine my free time. Then add to that the Lion upgrade which requires and
Xcode upgrade (which may or may not compile QS with the release config,
etc.)

Short version: maybe tomorrow. :)

Reply to this email directly or view it on GitHub:
#439 (comment)

Owner

skurfer commented Jul 25, 2011

Xcode 4 Tangent

I think I’m testing this code, but really, I have no idea. Do I have to go to “Edit Scheme” every time I want to switch between Release and Debug?

Owner

skurfer commented Jul 25, 2011

Any idea why when pasting a URL into Mail.app, it ends up in a different font? (I know it’s opposed to be turned into a link, and it is, but seems like the font should match.) I won’t hold back the merge for this alone. Just curious.

Owner

pjrobertson commented Jul 25, 2011

I think I’m testing this code, but really, I have no idea. Do I have to go
to “Edit Scheme” every time I want to switch between Release and Debug?

I was reading somewhere, apparently Xcode 4 has things changed round to
'schemes'. What we'd need to do is make new schemes for debug and release
everything. It's something I want to bring up on the lists. I couldn't get a
release scheme to work (apparently if you go Product → Build → Archive it
makes a release scheme :S)

Any idea why when pasting a URL into Mail.app, it ends up in a different
font? (I know it’s opposed to be turned into a link, and it is, but seems
like the font should match.) I won’t hold back the merge for this alone.
Just curious.

Didn't pick up on that, but it doesn't look easy:
http://www.cocoabuilder.com/archive/cocoa/165465-nsattributedstring-gains-default-font-and-paragraph-style-after-round-tripping-through-pasteboard.html

I've done pretty much exactly what the OP did in that thread ^^.
RE the reply: it sounds overly complicated for what we want to do? :/

On 25 July 2011 04:30, skurfer <
reply@reply.github.com>wrote:

Any idea why when pasting a URL into Mail.app, it ends up in a different
font? (I know it’s opposed to be turned into a link, and it is, but seems
like the font should match.) I won’t hold back the merge for this alone.
Just curious.

Reply to this email directly or view it on GitHub:
#439 (comment)

@skurfer skurfer merged commit ade4798 into quicksilver:master Jul 25, 2011

Owner

pjrobertson commented Jul 25, 2011

Thanks for closing the issue Rob. I keep forgetting to put fixes #... in my commits :/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment