Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update XcodeCapp copyright notice in 'About' window #2774

Closed
wants to merge 1 commit into from

Conversation

enquora
Copy link
Contributor

@enquora enquora commented Sep 3, 2018

  • Use copyright string from Info.plist
    (was previously hard-coded in About window, despite also being set in Info.plist).
  • Create copyright property on AppDelegate.
  • Append 'All rights reserved' in code, remove from Info.plist value
    (make line breaking in About window easier to manage, ease localization, ease automated checking of copyright dates.)
  • Display copyright notice using bindings.
  • Resize copyright field to prevent truncated text.
  • Set resize masks for all window fields - not currently set.
  • Resize window to golden ratio for improved visual presentation.

Use copyright string from Info.plist
    (was previously hard-coded in About window, despite also being set in Info.plist).
Create copyright property on AppDelegate.
Append 'All rights reserved' in code, remove from Info.plist value
    (make line breaking in About window easier to manage, ease localization, ease automated checking of copyright dates.)
Display copyright notice using bindings.
Resize copyright field to prevent truncated text.
Set resize masks for all window fields - not currently set.
Resize window to golden ratio for improved visual presentation.
@cappbot cappbot added this to the Someday milestone Sep 3, 2018
@cappbot cappbot added the #new label Sep 3, 2018
@cappbot
Copy link

cappbot commented Sep 3, 2018

Milestone: Someday. Label: #new. What's next? A reviewer should examine this issue.

@cacaodev
Copy link
Contributor

@enquora I'm sorry but it seems we've created some conflicts with this pr. Can you resolve them, otherwise it seems good to me. Thank you for this.

@enquora
Copy link
Contributor Author

enquora commented Sep 25, 2018

I may well have implemented this prior to the full merge of 1.0. Will clean it up — nothing dramatic is changed.

@daboe01
Copy link
Contributor

daboe01 commented Sep 26, 2018

-#new
+#needs-improvement

@cappbot
Copy link

cappbot commented Sep 26, 2018

Milestone: Someday. Label: #needs-improvement. What's next? The code for this issue has problems with formatting or fails a capp_lint check, has bugs, or has non-optimal logic or algorithms. It should be improved upon.

@mrcarlberg
Copy link
Member

@enquora Can you fix the conflicts on this pull request so it can be merged? Also the test cases fails for some reason.

@enquora
Copy link
Contributor Author

enquora commented Jul 5, 2019

@mrcarlberg Someone posted a link to dark mode support several weeks ago in Gitter. This should probably be fixed simultaneously with that but I'm having trouble locating it in the Gitter history and don't see a pull request. You don't remember roughly when that was, do you?

@daboe01
Copy link
Contributor

daboe01 commented Apr 1, 2022

@enquora can you please update this one by re-running nib2cib? this should fix the merge-conflict.

@daboe01
Copy link
Contributor

daboe01 commented Jun 2, 2024

@enquora is this PR possibly obsoleted with your recent work on XcodeCapp?

@enquora
Copy link
Contributor Author

enquora commented Jun 2, 2024

It's intended to keep this separate from the major work on XcodeCapp.
This PR is being used as a sort of bookmark for myself while my local CappuccinoSource is cleaned up to reflect the promotion of Node to default.

I would like to keep this open for another week or two while this and a few other items are prepped for (re)submitting.

@enquora enquora closed this Jun 19, 2024
@enquora
Copy link
Contributor Author

enquora commented Jun 19, 2024

Replaced with new commits to 'main' branch only.

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

Successfully merging this pull request may close these issues.

None yet

5 participants