-
Notifications
You must be signed in to change notification settings - Fork 22
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
Image paths to be corrected #18
Comments
History marcelklehr/socialcalc was split out to add OTs. It used to be part of audreyt/ethercalc Constants Old https://github.com/marcelklehr/socialcalc/blob/master/socialcalcconstants.js#L144 |
Thanks, but i noticed the error because of a console log. When i changed the values in the Constants file it fixed things. Are we sure this repo is up to date? |
Do you want me to re-open? The correct values exist in ethercalc, but you are welcome to fix things here if you want. |
I guess this could be considered a duplicate of #4, which would be a good improvement, imo. I think this repo should contain everything necessary or directly useful to running socialcalc in a different environment than ethercalc. |
Agreed, and right now there are extra steps to make it work as expected. For instance, the sheet starts in what seems to be an unstable mode where we can only edit cell A1. We have to programmatically select tab "edit" to get normal behavior. My concern right now is that we have to do a lot of work to make it have a normal usable scenario, and will start to use our choices of styling and frameworks -- so we will have our own fork which may not be compatible with your design choices for ethercalc. For example, the CSS is global (I would expect all the CSS selectors to be preceded with something like |
I advocate for not forking. What do you mean by design choices? I'm deferring to @audreyt on this, obviously, but my take is that probably anything you would do to improve socialcalc would benefit ethercalc as well and they would be happy to integrate those changes, the only problem being that someone has to update ethercalc to use the new APIs and stuff, but as there's not much else happening here atm, afaik, I'd say ethercalc could link to a fixed version of socialcalc and we could pull any pull requests you're shooting at us. It might even make sense to simply have a new version branch for the new features and fixes and keep the 2.x branch open for hotfixes and stuff. |
I missed this comment above by @marcelklehr The problem is that without a framework there is so much duplication of code and serious dissimilarity in look and feel (for example: the popups, buttons, icons etc.) when you include in an app -- which is what we are doing. Let me set up our app with SocialCalc built-in and send you credentials and you can judge. |
In the constants file, there are many references to
sc-<something>.gif
, when the images folder followssc_<something>.gif
-- Just documenting, feel free to ignore
The text was updated successfully, but these errors were encountered: